Changeset 230616 in webkit

Timestamp:

Apr 12, 2018 5:11:06 PM (6 years ago)

Author:

dbates@webkit.org

Message:

Attmept to fix the Windows build following <​https://trac.webkit.org/changeset/230602/>
(​https://bugs.webkit.org/show_bug.cgi?id=184386)

Visual Studio cannot prove that all cases in the switch are covered :(
So, add an ASSERT_NOT_REACHED() and a return a value outside of the switch block.

• loader/cache/CachedResourceLoader.cpp:

(WebCore::destinationForType):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• loader/cache/CachedResourceLoader.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2018-04-12  Daniel Bates  <dabates@apple.com>
+
+        Attmept to fix the Windows build following <https://trac.webkit.org/changeset/230602/>
+        (https://bugs.webkit.org/show_bug.cgi?id=184386)
+
+        Visual Studio cannot prove that all cases in the switch are covered :(
+        So, add an ASSERT_NOT_REACHED() and a return a value outside of the switch block. 
+
+        * loader/cache/CachedResourceLoader.cpp:
+        (WebCore::destinationForType):
+
 2018-04-12  Chris Dumez  <cdumez@apple.com>
 

trunk/Source/WebCore/loader/cache/CachedResourceLoader.cpp

@@ -767 +767 @@
         return FetchOptions::Destination::EmptyString;
     }
+    ASSERT_NOT_REACHED();
+    return FetchOptions::Destination::EmptyString;
 }