Merge pull request #218 from ethereum/display-warnings

Display warnings even if there were no errors

Merge pull request #218 from ethereum/display-warnings
22861f70 · chriseth · GitHub · ebe8b2e3 · 498ac727 · 22861f70
Commit 22861f70 authored Sep 07, 2016 by chriseth Committed by GitHub Sep 07, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 10 additions and 9 deletions

renderer.js src/app/renderer.js +10 -9

No files found.
--- a/src/app/renderer.js
+++ b/src/app/renderer.js
@@ -19,15 +19,16 @@ function Renderer (editor, web3, updateFiles, udapp, executionContext, formalVer
    $('#output').empty();
    if (success) {
      self.contracts(data, source);
-    } else {
+    }
-      if (data['error']) {
-        self.error(data['error']);
+    // NOTE: still need to display as there might be warnings
-      }
+    if (data['error']) {
-      if (data['errors']) {
+      self.error(data['error']);
-        data['errors'].forEach(function (err) {
+    }
-          self.error(err);
+    if (data['errors']) {
-        });
+      data['errors'].forEach(function (err) {
-      }
+        self.error(err);
+      });
    }
  });
 }