Build fix for newer clang
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 25 Feb 2017 00:20:12 +0000 (00:20 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 25 Feb 2017 00:20:12 +0000 (00:20 +0000)
https://bugs.webkit.org/show_bug.cgi?id=168849
rdar://problem/30638741

Reviewed by Ryosuke Niwa.

* dom/Range.cpp: (WebCore::Range::toString): std::max(0, <unsigned>) is not meaningful,
and now triggers a warning.

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

Source/WebCore/ChangeLog
Source/WebCore/dom/Range.cpp

index 7336663..80995c9 100644 (file)
@@ -1,3 +1,14 @@
+2017-02-24  Alexey Proskuryakov  <ap@apple.com>
+
+        Build fix for newer clang
+        https://bugs.webkit.org/show_bug.cgi?id=168849
+        rdar://problem/30638741
+
+        Reviewed by Ryosuke Niwa.
+
+        * dom/Range.cpp: (WebCore::Range::toString): std::max(0, <unsigned>) is not meaningful,
+        and now triggers a warning.
+
 2017-02-24  Chris Dumez  <cdumez@apple.com>
 
         Download attribute should be sanitized before being used as suggested filename
index e46266a..e0ed748 100644 (file)
@@ -944,7 +944,7 @@ String Range::toString() const
         if (type == Node::TEXT_NODE || type == Node::CDATA_SECTION_NODE) {
             auto& data = downcast<CharacterData>(*node).data();
             unsigned length = data.length();
-            unsigned start = node == &startContainer() ? std::min(std::max(0U, m_start.offset()), length) : 0U;
+            unsigned start = node == &startContainer() ? std::min(m_start.offset(), length) : 0U;
             unsigned end = node == &endContainer() ? std::min(std::max(start, m_end.offset()), length) : length;
             builder.append(data, start, end - start);
         }