From c1f540a2ebfeefedca699c1c42722ac14e1208fc Mon Sep 17 00:00:00 2001 From: Markus Scherer Date: Wed, 12 Mar 2014 18:48:27 +0000 Subject: [PATCH] ICU-10640 remove obsolete TODO X-SVN-Rev: 35435 --- icu4c/source/i18n/measfmt.cpp | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/icu4c/source/i18n/measfmt.cpp b/icu4c/source/i18n/measfmt.cpp index e3210cf351e..a0de39534dc 100644 --- a/icu4c/source/i18n/measfmt.cpp +++ b/icu4c/source/i18n/measfmt.cpp @@ -374,8 +374,8 @@ static UBool getFromCache( } static UBool isTimeUnit(const MeasureUnit &mu, const char *tu) { - return uprv_strcmp(mu.getType(), "duration") == 0 && - uprv_strcmp(mu.getSubtype(), tu) == 0; + return uprv_strcmp(mu.getType(), "duration") == 0 && + uprv_strcmp(mu.getSubtype(), tu) == 0; } // Converts a composite measure into hours-minutes-seconds and stores at hms @@ -400,9 +400,6 @@ static int32_t toHMS( // We use copy constructor to ensure that both sides of equality operator // are instances of MeasureUnit base class and not a subclass. Otherwise, // operator== will immediately return false. - // TODO(Travis Keep): Propose - // UBool isUnitSame(const MeasureUnit &other) const to address performance - // issues around copy constructor. for (int32_t i = 0; i < measureCount; ++i) { if (isTimeUnit(measures[i].getUnit(), "hour")) { // hour must come first -- 2.40.0