commit a4b53d0da8c1c6889c361fd30b913adc364163bc
author Yifan Hong <elsk@google.com> Mon Oct 31 17:29:10 2016 -0700
committer Yifan Hong <elsk@google.com> Thu Nov 03 09:39:11 2016 -0700
tree 0179a0eb9d796f868ceae09c3036bf09adf5ccfa
parent 447616f2f8b980849b4abc3b4d1a4db1ae00d3f7

Add location info to named types.

Bug: 31827278

Test: hidl_test
Change-Id: I1922d5c309c9706cef37ab34d28e1193d1e247cd

AST.cpp

diff --git a/AST.cpp b/AST.cpp
index f61ad4f..f9a6515 100644
--- a/AST.cpp
+++ b/AST.cpp
@@ -21,6 +21,7 @@
 #include "FQName.h"
 #include "HandleType.h"
 #include "Interface.h"
+#include "Location.h"
 #include "PredefinedType.h"
 #include "Scope.h"
 #include "TypeDef.h"
@@ -36,7 +37,7 @@
     : mCoordinator(coordinator),
       mPath(path),
       mScanner(NULL),
-      mRootScope(new Scope("" /* localName */)) {
+      mRootScope(new Scope("" /* localName */, Location::startOf(path))) {
     enterScope(mRootScope);
 }
 
@@ -188,13 +189,13 @@
     return mScopePath.back();
 }
 
-bool AST::addTypeDef(
-        const char *localName, Type *type, std::string *errorMsg) {
+bool AST::addTypeDef(const char *localName, Type *type, const Location &location,
+        std::string *errorMsg) {
     // The reason we wrap the given type in a TypeDef is simply to suppress
     // emitting any type definitions later on, since this is just an alias
     // to a type defined elsewhere.
     return addScopedTypeInternal(
-            new TypeDef(localName, type), errorMsg);
+            new TypeDef(localName, location, type), errorMsg);
 }
 
 bool AST::addScopedType(NamedType *type, std::string *errorMsg) {