cleanup main

src/main.cpp

Index: src/main.cpp
diff --git a/src/main.cpp b/src/main.cpp
index fa5bb069442a39baf351fa3c1f97daf56741593c..bef3d84fbc7097ba5756eb91f76e3f55dd01c0a1 100644
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -14,7 +14,6 @@
 //===----------------------------------------------------------------------===//
 
 #include "IceBrowserCompileServer.h"
-#include "IceCompiler.h"
 #include "IceCompileServer.h"
 
 int main(int argc, char **argv) {
@@ -28,11 +27,9 @@ int main(int argc, char **argv) {
   (void)argc;
   (void)argv;
   Ice::BrowserCompileServer Server(Comp);
-  Server.run();
-  return Server.getErrorCode().value();
 #else  // !PNACL_BROWSER_TRANSLATOR

[Jim Stichnoth, 2015/12/04 18:09:05]

We've generally tried to avoid conditional (#if) compilation as much as
possible, via the constexpr functions in Ice::BuildDefs.  It looks like this
instance may have slipped through.

Can you try adding a function like Ice::BuildDefs::browser() that returns
PNACL_BROWSER_TRANSLATOR, and use that to guard the code here?

   Ice::CLCompileServer Server(Comp, argc, argv);
+#endif // !PNACL_BROWSER_TRANSLATOR
   Server.run();
   return Server.getErrorCode().value();
-#endif // !PNACL_BROWSER_TRANSLATOR
 }