new version: 2017.1.0

simgear/package/CatalogTest.cxx

@@ -175,14 +175,9 @@ int parseTest()
       // expected
     }
 
-    unsigned int skisVariantFull = p2->indexOfVariant("org.flightgear.test.catalog1.c172p-skis");
-    VERIFY(skisVariantFull > 0);
-
     unsigned int skisVariant = p2->indexOfVariant("c172p-skis");
     VERIFY(skisVariant > 0);
 
-    COMPARE(skisVariant, skisVariantFull);
-
     pkg::Package::ThumbnailVec thumbs2 = p2->thumbnailsForVariant(skisVariant);
     COMPARE(thumbs2.size(), 2);
 

simgear/package/Package.cxx

@@ -340,25 +340,13 @@ std::string Package::nameForVariant(const std::string& vid) const
 
 unsigned int Package::indexOfVariant(const std::string& vid) const
 {
-    // accept fully-qualified IDs here
-    std::string actualId = vid;
-    size_t lastDot = vid.rfind('.');
-    if (lastDot != std::string::npos) {
-        std::string catalogId = vid.substr(0, lastDot);
-        if (catalogId != catalog()->id()) {
-            throw sg_exception("Bad fully-qualified ID:" + vid + ", package mismatch" );
-        }
-        actualId = vid.substr(lastDot + 1);
-    }
-
-
-    if (actualId == id()) {
+    if (vid == id()) {
         return 0;
     }
 
     unsigned int result = 1;
     for (SGPropertyNode* var : m_props->getChildren("variant")) {
-        if (var->getStringValue("id") == actualId) {
+        if (var->getStringValue("id") == vid) {
             return result;
         }
 

version

@@ -1 +1 @@
-2016.4.3
+2017.1.0