]> granicus.if.org Git - icu/commitdiff
Fix comments: remove a WIP, fix expectations in disabled/failing test.
authorHugo van der Merwe <17109322+hugovdm@users.noreply.github.com>
Mon, 30 Mar 2020 12:14:02 +0000 (14:14 +0200)
committerHugo van der Merwe <17109322+hugovdm@users.noreply.github.com>
Mon, 30 Mar 2020 12:49:56 +0000 (14:49 +0200)
icu4c/source/i18n/unitsdata.cpp
icu4c/source/test/intltest/unitsdatatest.cpp

index b26f6f557993fb2391e85d1cec6c1bdae71fb840..5daf170cf2d07546ed8bc5346dec28b46ca83203 100644 (file)
@@ -343,7 +343,6 @@ MaybeStackVector<ConversionRateInfo> getConversionRatesInfo(const MeasureUnit so
         MeasureUnit baseUnit;
         processSingleUnit(targetUnits[i], convertUnitsBundle.getAlias(), convertSink, &baseUnit, status);
         if (target.getComplexity(status) == UMEASURE_UNIT_SEQUENCE) {
-            // WIP/TODO(hugovdm): add consistency checks.
             if (baseUnit != sourceBaseUnit) {
                 status = U_ILLEGAL_ARGUMENT_ERROR;
                 return result;
@@ -364,6 +363,8 @@ MaybeStackVector<ConversionRateInfo> getConversionRatesInfo(const MeasureUnit so
             // fprintf(stderr, "<%s> x <%s> => ",
             //         targetBaseUnit.getIdentifier(),
             //         baseUnit.getIdentifier());
+            //
+            // Expected: <kilogram-square-meter-per-meter-square-second>
             targetBaseUnit = targetBaseUnit.product(baseUnit, status);
             // fprintf(stderr, "<%s> - Status: %s\n",
             //         targetBaseUnit.getIdentifier(), u_errorName(status));
index e5c510160091e01af07ad14abdb15946f6e2c9a1..32fa616f8fddf018ca41a4dd250ee8bc0ed094e0 100644 (file)
@@ -70,7 +70,7 @@ void UnitsDataTest::testGetConversionRateInfo() {
         // {"therm-us-per-meter",
         //  "joule-per-meter",
         //  {"therm-us", "joule", "meter", NULL, NULL},
-        //  "kilogram-meter-per-square-second"},
+        //  "kilogram-square-meter-per-square-second"},
 
         // TODO: include capacitance test case with base unit:
         // pow4-second-square-ampere-per-kilogram-square-meter;