2010-08-12 Csaba Osztrogon√°c <ossy@webkit.org>
authorossy@webkit.org <ossy@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Aug 2010 21:55:44 +0000 (21:55 +0000)
committerossy@webkit.org <ossy@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Aug 2010 21:55:44 +0000 (21:55 +0000)
        Reviewed by Darin Adler.

        generate-bindings.pl should generate warning free code
        https://bugs.webkit.org/show_bug.cgi?id=43353

        * bindings/scripts/CodeGeneratorJS.pm: Suggested parentheses added around && within ||

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@65269 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/bindings/scripts/CodeGeneratorJS.pm

index 4361419..64f892c 100644 (file)
@@ -1,3 +1,12 @@
+2010-08-12  Csaba Osztrogon√°c  <ossy@webkit.org>
+
+        Reviewed by Darin Adler.
+
+        generate-bindings.pl should generate warning free code
+        https://bugs.webkit.org/show_bug.cgi?id=43353
+
+        * bindings/scripts/CodeGeneratorJS.pm: Suggested parentheses added around && within ||
+
 2010-08-11  Ryosuke Niwa  <rniwa@webkit.org>
 
         Reviewed by Kent Tamura.
index cb7c333..f48159b 100644 (file)
@@ -1135,7 +1135,7 @@ sub GenerateParametersCheckExpression
         # these are acceptable values for a DOMString argument (any Object can
         # be converted to a string via .toString).
         push(@andExpression, "(${value}.isNull() || ${value}.isUndefined() || ${value}.isString() || ${value}.isObject())") if $codeGenerator->IsStringType($type);
-        push(@andExpression, "(${value}.isNull() || ${value}.isObject() && asObject(${value})->inherits(&JS${type}::s_info))") unless IsNativeType($type);
+        push(@andExpression, "(${value}.isNull() || (${value}.isObject() && asObject(${value})->inherits(&JS${type}::s_info)))") unless IsNativeType($type);
 
         $parameterIndex++;
     }