commit b0d15a5ca155edff29a020c66fb5c3f8af93ded9
author Steven Moreland <smoreland@google.com> Tue Mar 31 14:03:47 2020 -0700
committer Steven Moreland <smoreland@google.com> Wed Apr 01 16:49:25 2020 +0000
tree 5f2c804842adeec68fcae5fadaf6eabbcb625149
parent d131d5eb12b71a9ae5e22019248e79102c871ce1

Distinguish enum and class 'AidlError'.

These are in different namespaces, but it is still confusing and makes
refactoring more difficult. Renaming the class to AidlErrorLog.

Bug: N/A
Test: compilation suffices
Change-Id: Ib4d4e8c25d87756f182ff120feb9825f58a6d30e

aidl_language.h

diff --git a/aidl_language.h b/aidl_language.h
index 97c8ba3..86640e9 100644
--- a/aidl_language.h
+++ b/aidl_language.h
@@ -110,7 +110,7 @@
   virtual ~AidlNode() = default;
 
   // To be able to print AidlLocation
-  friend class AidlError;
+  friend class AidlErrorLog;
   friend std::string android::aidl::mappings::dump_location(const AidlNode&);
   friend std::string android::aidl::java::dump_location(const AidlNode&);
 
@@ -125,16 +125,18 @@
 };
 
 // Generic point for printing any error in the AIDL compiler.
-class AidlError {
+class AidlErrorLog {
  public:
-  AidlError(bool fatal, const std::string& filename) : AidlError(fatal) { os_ << filename << ": "; }
-  AidlError(bool fatal, const AidlLocation& location);
-  AidlError(bool fatal, const AidlNode& node) : AidlError(fatal, node.location_) {}
-  AidlError(bool fatal, const AidlNode* node) : AidlError(fatal, *node) {}
+  AidlErrorLog(bool fatal, const std::string& filename) : AidlErrorLog(fatal) {
+    os_ << filename << ": ";
+  }
+  AidlErrorLog(bool fatal, const AidlLocation& location);
+  AidlErrorLog(bool fatal, const AidlNode& node) : AidlErrorLog(fatal, node.location_) {}
+  AidlErrorLog(bool fatal, const AidlNode* node) : AidlErrorLog(fatal, *node) {}
 
   template <typename T>
-  AidlError(bool fatal, const std::unique_ptr<T>& node) : AidlError(fatal, *node) {}
-  ~AidlError() {
+  AidlErrorLog(bool fatal, const std::unique_ptr<T>& node) : AidlErrorLog(fatal, *node) {}
+  ~AidlErrorLog() {
     os_ << std::endl;
     if (fatal_) abort();
   }
@@ -144,17 +146,17 @@
   static bool hadError() { return sHadError; }
 
  private:
-  AidlError(bool fatal);
+  AidlErrorLog(bool fatal);
 
   bool fatal_;
 
   static bool sHadError;
 
-  DISALLOW_COPY_AND_ASSIGN(AidlError);
+  DISALLOW_COPY_AND_ASSIGN(AidlErrorLog);
 };
 
-#define AIDL_ERROR(CONTEXT) ::AidlError(false /*fatal*/, (CONTEXT)).os_
-#define AIDL_FATAL(CONTEXT) ::AidlError(true /*fatal*/, (CONTEXT)).os_
+#define AIDL_ERROR(CONTEXT) ::AidlErrorLog(false /*fatal*/, (CONTEXT)).os_
+#define AIDL_FATAL(CONTEXT) ::AidlErrorLog(true /*fatal*/, (CONTEXT)).os_
 #define AIDL_FATAL_IF(CONDITION, CONTEXT) \
   if (CONDITION) AIDL_FATAL(CONTEXT) << "Bad internal state: " << #CONDITION << ": "