[Qt] Build fix: warning on L933 of CompositeEditCommand.cpp
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 21 Apr 2010 07:46:30 +0000 (07:46 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 21 Apr 2010 07:46:30 +0000 (07:46 +0000)
https://bugs.webkit.org/show_bug.cgi?id=37912

Reviewed by Eric Seidel.

Replaced the ternary operator by an if statement because GCC was confused by the use of
ternary operator and producing warnings on Qt builds.

* WebCore.xcodeproj/project.pbxproj:
* editing/CompositeEditCommand.cpp:
(WebCore::CompositeEditCommand::moveParagraphs):

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

WebCore/ChangeLog
WebCore/editing/CompositeEditCommand.cpp

index 711bd16..5ebc13f 100644 (file)
@@ -1,5 +1,19 @@
 2010-04-20  Ryosuke Niwa  <rniwa@webkit.org>
 
+        Reviewed by Eric Seidel.
+
+        [Qt] Build fix: warning on L933 of CompositeEditCommand.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=37912
+
+        Replaced the ternary operator by an if statement because GCC was confused by the use of
+        ternary operator and producing warnings on Qt builds.
+
+        * WebCore.xcodeproj/project.pbxproj:
+        * editing/CompositeEditCommand.cpp:
+        (WebCore::CompositeEditCommand::moveParagraphs):
+
+2010-04-20  Ryosuke Niwa  <rniwa@webkit.org>
+
         Reviewed by Justin Garcia.
 
         Nested <ul>s are mishandled when converted to <ol> using execCommand('insertorderedlist')
index fff76a6..9dc918d 100644 (file)
@@ -930,8 +930,11 @@ void CompositeEditCommand::moveParagraphs(const VisiblePosition& startOfParagrap
 
     // FIXME: This is an inefficient way to preserve style on nodes in the paragraph to move. It
     // shouldn't matter though, since moved paragraphs will usually be quite small.
-    RefPtr<DocumentFragment> fragment = startOfParagraphToMove != endOfParagraphToMove ? createFragmentFromMarkup(document(), createMarkup(range.get(), 0, DoNotAnnotateForInterchange, true), "") : 0;
-    
+    RefPtr<DocumentFragment> fragment;
+    // This used to use a ternary for initialization, but that confused some versions of GCC, see bug 37912
+    if (startOfParagraphToMove != endOfParagraphToMove)
+        fragment = createFragmentFromMarkup(document(), createMarkup(range.get(), 0, DoNotAnnotateForInterchange, true), "");
+
     // A non-empty paragraph's style is moved when we copy and move it.  We don't move 
     // anything if we're given an empty paragraph, but an empty paragraph can have style
     // too, <div><b><br></b></div> for example.  Save it so that we can preserve it later.