From bb739369599e374295291ea41d9fbcb5ee3c78b1 Mon Sep 17 00:00:00 2001
From: Pavol Rusnak <stick@gk2.sk>
Date: Mon, 7 Jul 2014 21:34:54 +0200
Subject: [PATCH] use more warnings from trezor-mcu

---
 Makefile       | 31 +++++++++++++++++++++++++++++--
 test-openssl.c |  2 +-
 tests.c        |  2 +-
 3 files changed, 31 insertions(+), 4 deletions(-)

diff --git a/Makefile b/Makefile
index e6029f404..db65f6776 100644
--- a/Makefile
+++ b/Makefile
@@ -1,8 +1,35 @@
-CC     = gcc
-CFLAGS = -Os -Wall -Wextra -Wno-sequence-point -Wundef
+CC        = gcc
+
+OPTFLAGS  = -Os -g
+
+CFLAGS   += $(OPTFLAGS) \
+            -W \
+            -Wall \
+            -Wextra \
+            -Wimplicit-function-declaration \
+            -Wredundant-decls \
+            -Wstrict-prototypes \
+            -Wundef \
+            -Wshadow \
+            -Wpointer-arith \
+            -Wformat \
+            -Wreturn-type \
+            -Wsign-compare \
+            -Wmultichar \
+            -Wformat-nonliteral \
+            -Winit-self \
+            -Wuninitialized \
+            -Wformat-security \
+            -Werror
+
+# disable sequence point warning because of AES code
+CFLAGS += -Wno-sequence-point
+
+# disable certain optimizations and features when small footprint is required
 ifdef SMALL
 CFLAGS += -DUSE_PRECOMPUTED_IV=0 -DUSE_PRECOMPUTED_CP=0 -DUSE_PUBKEY_VALIDATE=0
 endif
+
 OBJS   = bignum.o ecdsa.o secp256k1.o rand.o hmac.o bip32.o bip39.o pbkdf2.o base58.o
 OBJS  += ripemd160.o
 OBJS  += sha2.o
diff --git a/test-openssl.c b/test-openssl.c
index 696563833..ef36b9eea 100644
--- a/test-openssl.c
+++ b/test-openssl.c
@@ -30,7 +30,7 @@
 #include "ecdsa.h"
 #include "rand.h"
 
-int main()
+int main(void)
 {
 	uint8_t sig[64], pub_key33[33], pub_key65[65], priv_key[32], msg[256], buffer[1000], hash[32], *p;
 	uint32_t i, j, msg_len;
diff --git a/tests.c b/tests.c
index b6d818d27..d35655c43 100644
--- a/tests.c
+++ b/tests.c
@@ -1090,7 +1090,7 @@ Suite *test_suite(void)
 }
 
 // run suite
-int main()
+int main(void)
 {
 	int number_failed;
 	Suite *s = test_suite();