commit 70a59e1dc3dcf32f791d2dd7966111d4adf32eca
author Andreas Huber <andih@google.com> Tue Aug 16 12:57:01 2016 -0700
committer Andreas Huber <andih@google.com> Tue Aug 16 13:02:13 2016 -0700
tree 72dc0677ff37d23f4a7ec6290c03c6f9ffbd2aa2
parent c7dfef3fda2061dcaeeb144c10eef3cf7360b03c

Validate upfront that an interface is compatible with our Java backend,

emit diagnostics if it is not. Also cleans up all other error messages to
clearly indicate an error condition.

Bug: 30876839
Change-Id: I18bcd723107ab93abcad38c976f3c38dda60a743

main.cpp

diff --git a/main.cpp b/main.cpp
index 50edcae..2c79bdb 100644
--- a/main.cpp
+++ b/main.cpp
@@ -39,7 +39,7 @@
 
     if (ast == NULL) {
         fprintf(stderr,
-                "Could not parse %s. Aborting.\n",
+                "ERROR: Could not parse %s. Aborting.\n",
                 fqName.string().c_str());
 
         return UNKNOWN_ERROR;
@@ -119,7 +119,7 @@
 
         if (ast == NULL) {
             fprintf(stderr,
-                    "Could not parse %s. Aborting.\n",
+                    "ERROR: Could not parse %s. Aborting.\n",
                     fqName.string().c_str());
 
             return UNKNOWN_ERROR;
@@ -213,18 +213,18 @@
 
 OutputHandler::ValRes validateForMakefile(const FQName &fqName) {
     if (fqName.package().empty()) {
-        fprintf(stderr, "Expecting package name\n");
+        fprintf(stderr, "ERROR: Expecting package name\n");
         return OutputHandler::FAILED;
     }
 
     if (fqName.version().empty()) {
-        fprintf(stderr, "Expecting package version\n");
+        fprintf(stderr, "ERROR: Expecting package version\n");
         return OutputHandler::FAILED;
     }
 
     if (!fqName.name().empty()) {
         fprintf(stderr,
-                "Expecting only package name and version.\n");
+                "ERROR: Expecting only package name and version.\n");
         return OutputHandler::FAILED;
     }
 
@@ -233,12 +233,12 @@
 
 OutputHandler::ValRes validateForSource(const FQName &fqName) {
     if (fqName.package().empty()) {
-        fprintf(stderr, "Expecting package name\n");
+        fprintf(stderr, "ERROR: Expecting package name\n");
         return OutputHandler::FAILED;
     }
 
     if (fqName.version().empty()) {
-        fprintf(stderr, "Expecting package version\n");
+        fprintf(stderr, "ERROR: Expecting package version\n");
         return OutputHandler::FAILED;
     }
 
@@ -432,7 +432,7 @@
 
         if (!fqName.isValid()) {
             fprintf(stderr,
-                    "Invalid fully-qualified name.\n");
+                    "ERROR: Invalid fully-qualified name.\n");
             exit(1);
         }