Title: [92549] trunk/Source/WebCore
Revision
92549
Author
[email protected]
Date
2011-08-06 03:02:38 -0700 (Sat, 06 Aug 2011)

Log Message

Should use C++-style cast in CSSParser.cpp
https://bugs.webkit.org/show_bug.cgi?id=65807

Replaces C-style cast with C++style cast.

Reviewed by Shinichiro Hamaji.

No new tests because no behavior change.

* css/CSSParser.cpp:
(WebCore::CSSParser::parseAnimationTimingFunction):
(WebCore::CSSParser::parseFont):
(WebCore::CSSParser::parseColorFromValue):
(WebCore::CSSParser::parseFontFeatureTag):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (92548 => 92549)


--- trunk/Source/WebCore/ChangeLog	2011-08-06 09:40:43 UTC (rev 92548)
+++ trunk/Source/WebCore/ChangeLog	2011-08-06 10:02:38 UTC (rev 92549)
@@ -1,3 +1,20 @@
+2011-08-06  Kenichi Ishibashi  <[email protected]>
+
+        Should use C++-style cast in CSSParser.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=65807
+
+        Replaces C-style cast with C++style cast.
+
+        Reviewed by Shinichiro Hamaji.
+
+        No new tests because no behavior change.
+
+        * css/CSSParser.cpp:
+        (WebCore::CSSParser::parseAnimationTimingFunction):
+        (WebCore::CSSParser::parseFont):
+        (WebCore::CSSParser::parseColorFromValue):
+        (WebCore::CSSParser::parseFontFeatureTag):
+
 2011-08-05  Noel Gordon  <[email protected]>
 
         [Chromium] Remove HaltablePlugin references from gyp project files

Modified: trunk/Source/WebCore/css/CSSParser.cpp (92548 => 92549)


--- trunk/Source/WebCore/css/CSSParser.cpp	2011-08-06 09:40:43 UTC (rev 92548)
+++ trunk/Source/WebCore/css/CSSParser.cpp	2011-08-06 10:02:38 UTC (rev 92549)
@@ -3284,7 +3284,7 @@
         CSSParserValue* v = args->current();
         if (!validUnit(v, FInteger, m_strict))
             return 0;
-        numSteps = (int) min(v->fValue, (double)INT_MAX);
+        numSteps = static_cast<int>(min(v->fValue, (double)INT_MAX));
         if (numSteps < 1)
             return 0;
         v = args->next();
@@ -3929,7 +3929,7 @@
                 valid = false;
             }
         } else if (!font->weight && validUnit(value, FInteger | FNonNeg, true)) {
-            int weight = (int)value->fValue;
+            int weight = static_cast<int>(value->fValue);
             int val = 0;
             if (weight == 100)
                 val = CSSValue100;
@@ -4764,7 +4764,7 @@
 {
     if (!m_strict && value->unit == CSSPrimitiveValue::CSS_NUMBER &&
         value->fValue >= 0. && value->fValue < 1000000.) {
-        String str = String::format("%06d", (int)(value->fValue+.5));
+        String str = String::format("%06d", static_cast<int>((value->fValue+.5)));
         if (!CSSParser::parseColor(str, c, m_strict))
             return false;
     } else if (value->unit == CSSPrimitiveValue::CSS_PARSER_HEXCOLOR ||
@@ -6333,7 +6333,7 @@
     value = m_valueList->next();
     if (value) {
         if (value->unit == CSSPrimitiveValue::CSS_NUMBER && value->isInt && value->fValue >= 0) {
-            tagValue = (int)value->fValue;
+            tagValue = static_cast<int>(value->fValue);
             if (tagValue < 0)
                 return false;
             m_valueList->next();
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to