commit 90ea87f36e60a8db0c12d8e7870d45c90c51922d
author Yifan Hong <elsk@google.com> Tue Nov 01 14:25:47 2016 -0700
committer Yifan Hong <elsk@google.com> Wed Nov 02 16:29:00 2016 +0000
tree 8a5c4b66a6cea1f1161e7a5fc752c2286b829cfb
parent ee88eed2002a2becda0b070cdf8900596a92ba53

Remove "@" from FQName.mVersion.

FQName::version() no longer returns the version
with the "@" prefix.

Removed FQName::getPackageFullVersion(). Use
FQName::version() instead.

Test: mma
Test: hidl_test
Bug: 31914582
Change-Id: Iaf05dd17aa338ebb631805badaabcf5abf1b261e

Coordinator.cpp

diff --git a/Coordinator.cpp b/Coordinator.cpp
index 2e17b61..57da8bb 100644
--- a/Coordinator.cpp
+++ b/Coordinator.cpp
@@ -226,8 +226,7 @@
     packagePath.append(packageSuffix.substr(startPos));
     packagePath.append("/");
 
-    CHECK_EQ(fqName.version().find('@'), 0u);
-    packagePath.append(fqName.version().substr(1));
+    packagePath.append(fqName.version());
     packagePath.append("/");
 
     return packagePath;
@@ -296,7 +295,7 @@
 
     for (const auto &fileName : fileNames) {
         FQName subFQName(
-                package.package() + package.version() + "::" + fileName);
+                package.package() + package.atVersion() + "::" + fileName);
 
         if (!subFQName.isValid()) {
             LOG(WARNING)
@@ -304,7 +303,7 @@
                 << fileName
                 << "' in package "
                 << package.package()
-                << package.version();
+                << package.atVersion();
 
             continue;
         }