From e3c5ca02fde4282a3797be6ecea5a473ac8c3a1d Mon Sep 17 00:00:00 2001
From: JF Bastien <jfb@chromium.org>
Date: Tue, 22 Dec 2015 13:51:20 -0800
Subject: Fix warnings found by GCC

My previous patch addressed all LLVM warnings, this one addresses all the GCC ones as well (mostly signed / unsigned mix).

The patch also turns on -Wall -Werror.
---
 src/binaryen-shell.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'src/binaryen-shell.cpp')

diff --git a/src/binaryen-shell.cpp b/src/binaryen-shell.cpp
index 75a138c99..4c4e21f6f 100644
--- a/src/binaryen-shell.cpp
+++ b/src/binaryen-shell.cpp
@@ -181,7 +181,8 @@ int main(int argc, char **argv) {
   bool print_after = false;
   std::vector<std::string> passes;
 
-  for (size_t i = 1; i < argc; i++) {
+  assert(argc > 0 && "expect at least program name as an argument");
+  for (size_t i = 1, e = argc; i != e; i++) {
     char* curr = argv[i];
     if (curr[0] == '-') {
       std::string arg = curr;
-- 
cgit v1.2.3