Changeset 82809 in webkit

Timestamp:

Apr 4, 2011 2:20:45 AM (13 years ago)

Author:

thakis@chromium.org

Message:

2011-04-04 Nico Weber <​thakis@chromium.org>

Reviewed by Eric Seidel.

Prefer a using directive over qualified names in LocalizedNumberICU.cpp
​https://bugs.webkit.org/show_bug.cgi?id=57742

Requested by ap in ​https://bugs.webkit.org/show_bug.cgi?id=57715

• platform/text/LocalizedNumberICU.cpp: (WebCore::createFormatterForCurrentLocale): (WebCore::numberFormatter): (WebCore::parseLocalizedNumber): (WebCore::formatLocalizedNumber):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/text/LocalizedNumberICU.cpp (modified) (3 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-04-04  Nico Weber  <thakis@chromium.org>
+
+        Reviewed by Eric Seidel.
+
+        Prefer a using directive over qualified names in LocalizedNumberICU.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=57742
+
+        Requested by ap in https://bugs.webkit.org/show_bug.cgi?id=57715
+
+        * platform/text/LocalizedNumberICU.cpp:
+        (WebCore::createFormatterForCurrentLocale):
+        (WebCore::numberFormatter):
+        (WebCore::parseLocalizedNumber):
+        (WebCore::formatLocalizedNumber):
+
 2011-04-04  Andrey Adaikin  <aandrey@google.com>
 

trunk/Source/WebCore/platform/text/LocalizedNumberICU.cpp

@@ -38 +38 @@
 #include <wtf/PassOwnPtr.h>
 
+using namespace icu;
 using namespace std;
 
 namespace WebCore {
 
-static PassOwnPtr<icu::NumberFormat> createFormatterForCurrentLocale()
+static PassOwnPtr<NumberFormat> createFormatterForCurrentLocale()
 {
     UErrorCode status = U_ZERO_ERROR;
-    OwnPtr<icu::NumberFormat> formatter = adoptPtr(icu::NumberFormat::createInstance(status));
+    OwnPtr<NumberFormat> formatter = adoptPtr(NumberFormat::createInstance(status));
     return U_SUCCESS(status) ? formatter.release() : 0;
 }
 
 // This might return 0.
-static icu::NumberFormat* numberFormatter()
+static NumberFormat* numberFormatter()
 {
     ASSERT(isMainThread());
-    static icu::NumberFormat* formatter = createFormatterForCurrentLocale().leakPtr();
+    static NumberFormat* formatter = createFormatterForCurrentLocale().leakPtr();
     return formatter;
 }
@@ -61 +62 @@
     if (numberString.isEmpty())
         return numeric_limits<double>::quiet_NaN();
-    icu::NumberFormat* formatter = numberFormatter();
+    NumberFormat* formatter = numberFormatter();
     if (!formatter)
         return numeric_limits<double>::quiet_NaN();
-    icu::UnicodeString numberUnicodeString(numberString.characters(), numberString.length());
-    icu::Formattable result;
-    icu::ParsePosition position(0);
+    UnicodeString numberUnicodeString(numberString.characters(), numberString.length());
+    Formattable result;
+    ParsePosition position(0);
     formatter->parse(numberUnicodeString, result, position);
     if (position.getIndex() != numberUnicodeString.length())
@@ -77 +78 @@
 String formatLocalizedNumber(double number, unsigned fractionDigits)
 {
-    icu::NumberFormat* formatter = numberFormatter();
+    NumberFormat* formatter = numberFormatter();
     if (!formatter)
         return String();
-    icu::UnicodeString result;
+    UnicodeString result;
     formatter->setMaximumFractionDigits(clampToInteger(fractionDigits));
     formatter->format(number, result);