Title: [223334] trunk/Source/WebCore
- Revision
- 223334
- Author
- tpop...@redhat.com
- Date
- 2017-10-15 21:16:56 -0700 (Sun, 15 Oct 2017)
Log Message
DataTransfer.cpp triggers -Wunused-but-set-variable
https://bugs.webkit.org/show_bug.cgi?id=178209
Reviewed by Wenson Hsieh.
Use the ASSERT_UNUSED to silence it.
* dom/DataTransfer.cpp:
(WebCore::DataTransfer::filesFromPasteboardAndItemList const):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (223333 => 223334)
--- trunk/Source/WebCore/ChangeLog 2017-10-16 03:42:06 UTC (rev 223333)
+++ trunk/Source/WebCore/ChangeLog 2017-10-16 04:16:56 UTC (rev 223334)
@@ -1,3 +1,15 @@
+2017-10-16 Tomas Popela <tpop...@redhat.com>
+
+ DataTransfer.cpp triggers -Wunused-but-set-variable
+ https://bugs.webkit.org/show_bug.cgi?id=178209
+
+ Reviewed by Wenson Hsieh.
+
+ Use the ASSERT_UNUSED to silence it.
+
+ * dom/DataTransfer.cpp:
+ (WebCore::DataTransfer::filesFromPasteboardAndItemList const):
+
2017-10-15 Sam Weinig <s...@webkit.org>
[Settings] Split non-macro generated parts of Settings into SettingsBase base class
Modified: trunk/Source/WebCore/dom/DataTransfer.cpp (223333 => 223334)
--- trunk/Source/WebCore/dom/DataTransfer.cpp 2017-10-16 03:42:06 UTC (rev 223333)
+++ trunk/Source/WebCore/dom/DataTransfer.cpp 2017-10-16 04:16:56 UTC (rev 223334)
@@ -304,7 +304,8 @@
itemListContainsItems = true;
}
- ASSERT(!itemListContainsItems || !addedFilesFromPasteboard);
+ bool containsItemsAndFiles = itemListContainsItems && addedFilesFromPasteboard;
+ ASSERT_UNUSED(containsItemsAndFiles, !containsItemsAndFiles);
return files;
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes