Changeset 135345 in webkit

Timestamp:

Nov 20, 2012, 8:54:44 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

REGRESSION(r135263): Fix a wrong argument for udat_close
​https://bugs.webkit.org/show_bug.cgi?id=102856

Patch by Kunihiko Sakamoto <​ksakamoto@chromium.org> on 2012-11-20
Reviewed by Kent Tamura.

Fixed a bug that passed wrong pointer to udat_close.

No new tests. No changes in behavior.

• platform/text/LocaleICU.cpp:

(WebCore::LocaleICU::initializeDateTimeFormat): Close correct object.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/text/LocaleICU.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-11-20  Kunihiko Sakamoto  <ksakamoto@chromium.org>
+
+        REGRESSION(r135263): Fix a wrong argument for udat_close
+        https://bugs.webkit.org/show_bug.cgi?id=102856
+
+        Reviewed by Kent Tamura.
+
+        Fixed a bug that passed wrong pointer to udat_close.
+
+        No new tests. No changes in behavior.
+
+        * platform/text/LocaleICU.cpp:
+        (WebCore::LocaleICU::initializeDateTimeFormat): Close correct object.
+
 2012-11-20  Kentaro Hara  <haraken@chromium.org>
 

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

@@ -306 +306 @@
     UDateFormat* dateTimeFormatWithoutSeconds = openDateFormat(UDAT_SHORT, UDAT_SHORT);
     m_dateTimeFormatWithoutSeconds = getDateFormatPattern(dateTimeFormatWithoutSeconds);
-    udat_close(dateTimeFormatWithSeconds);
+    udat_close(dateTimeFormatWithoutSeconds);
 
     OwnPtr<Vector<String> > timeAMPMLabels = createLabelVector(m_mediumTimeFormat, UDAT_AM_PMS, UCAL_AM, 2);