JavaScriptCore:
authordarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 25 Jan 2009 20:27:41 +0000 (20:27 +0000)
committerdarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 25 Jan 2009 20:27:41 +0000 (20:27 +0000)
2009-01-25  Darin Adler  <darin@apple.com>

        Reviewed by Sam Weinig.

        Bug 23352: Turn on more compiler warnings in the Mac build
        https://bugs.webkit.org/show_bug.cgi?id=23352

        Fourth patch: Deal with the last few stray warnings.

        * parser/Parser.cpp: Only declare jscyyparse if it's not already declared.
        This makes both separate compilation and all-in-one compilation work with the
        -Wredundant-decls warning.

WebCore:

2009-01-25  Darin Adler  <darin@apple.com>

        Reviewed by Sam Weinig.

        Bug 23352: Turn on more compiler warnings in the Mac build
        https://bugs.webkit.org/show_bug.cgi?id=23352

        Fourth patch: Deal with the last few stray warnings.

        * rendering/RenderBlock.cpp:
        (WebCore::RenderBlock::firstLineBlock): Handle the const_cast a little better.
        Also make the local variable be a RenderBlock so the cast is closer to the code
        that guarantees it's correct.

        * rendering/SVGRenderTreeAsText.cpp:
        (WebCore::writeStyle): Put the enum into a local variable that's a built-in
        type before using it with TextStream, so we don't get a warning due to -Wsign-promo.

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

JavaScriptCore/ChangeLog
JavaScriptCore/parser/Parser.cpp
WebCore/ChangeLog
WebCore/rendering/RenderBlock.cpp
WebCore/rendering/SVGRenderTreeAsText.cpp

index cf160e1..d7a9246 100644 (file)
@@ -5,6 +5,19 @@
         Bug 23352: Turn on more compiler warnings in the Mac build
         https://bugs.webkit.org/show_bug.cgi?id=23352
 
+        Fourth patch: Deal with the last few stray warnings.
+
+        * parser/Parser.cpp: Only declare jscyyparse if it's not already declared.
+        This makes both separate compilation and all-in-one compilation work with the
+        -Wredundant-decls warning.
+
+2009-01-25  Darin Adler  <darin@apple.com>
+
+        Reviewed by Sam Weinig.
+
+        Bug 23352: Turn on more compiler warnings in the Mac build
+        https://bugs.webkit.org/show_bug.cgi?id=23352
+
         Third patch: Use the noreturn attribute on functions that don't
         return to prepare for the use of the -Wmissing-noreturn warning.
 
index 1b1e4d7..886a513 100644 (file)
@@ -31,7 +31,9 @@
 
 using std::auto_ptr;
 
+#ifndef yyparse
 extern int jscyyparse(void*);
+#endif
 
 namespace JSC {
 
index 728bdeb..ef67ee4 100644 (file)
@@ -2,6 +2,24 @@
 
         Reviewed by Sam Weinig.
 
+        Bug 23352: Turn on more compiler warnings in the Mac build
+        https://bugs.webkit.org/show_bug.cgi?id=23352
+
+        Fourth patch: Deal with the last few stray warnings.
+
+        * rendering/RenderBlock.cpp:
+        (WebCore::RenderBlock::firstLineBlock): Handle the const_cast a little better.
+        Also make the local variable be a RenderBlock so the cast is closer to the code
+        that guarantees it's correct.
+
+        * rendering/SVGRenderTreeAsText.cpp:
+        (WebCore::writeStyle): Put the enum into a local variable that's a built-in
+        type before using it with TextStream, so we don't get a warning due to -Wsign-promo.
+
+2009-01-25  Darin Adler  <darin@apple.com>
+
+        Reviewed by Sam Weinig.
+
         Bug 23522: use checked casts for render tree
         https://bugs.webkit.org/show_bug.cgi?id=23522
 
index c81bde2..84138b7 100644 (file)
@@ -4325,7 +4325,7 @@ bool RenderBlock::containsNonZeroBidiLevel() const
 
 RenderBlock* RenderBlock::firstLineBlock() const
 {
-    const RenderObject* firstLineBlock = this;
+    RenderBlock* firstLineBlock = const_cast<RenderBlock*>(this);
     bool hasPseudo = false;
     while (true) {
         hasPseudo = firstLineBlock->style()->hasPseudoStyle(RenderStyle::FIRST_LINE);
@@ -4335,13 +4335,14 @@ RenderBlock* RenderBlock::firstLineBlock() const
         if (firstLineBlock->isReplaced() || firstLineBlock->isFloating() || 
             !parentBlock || parentBlock->firstChild() != firstLineBlock || !parentBlock->isBlockFlow())
             break;
-        firstLineBlock = parentBlock;
+        ASSERT(parentBlock->isRenderBlock());
+        firstLineBlock = static_cast<RenderBlock*>(parentBlock);
     } 
     
     if (!hasPseudo)
         return 0;
     
-    return (RenderBlock*)(firstLineBlock);
+    return firstLineBlock;
 }
 
 void RenderBlock::updateFirstLetter()
index daeb82e..74a8af9 100644 (file)
@@ -233,8 +233,10 @@ static void writeStyle(TextStream& ts, const RenderObject& object)
 
     if (!object.localTransform().isIdentity())
         ts << " [transform=" << object.localTransform() << "]";
-    if (svgStyle->imageRendering() != SVGRenderStyle::initialImageRendering())
-        ts << " [image rendering=" << svgStyle->imageRendering() << "]";
+    if (svgStyle->imageRendering() != SVGRenderStyle::initialImageRendering()) {
+        unsigned imageRenderingAsInteger = svgStyle->imageRendering();
+        ts << " [image rendering=" << imageRenderingAsInteger << "]";
+    }
     if (style->opacity() != RenderStyle::initialOpacity())
         ts << " [opacity=" << style->opacity() << "]";
     if (object.isRenderPath()) {