Bug 159193 - [JSC] Fix build warning since r202502
Summary: [JSC] Fix build warning since r202502
Status: RESOLVED DUPLICATE of bug 159194
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Gyuyoung Kim
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-06-27 21:41 PDT by Gyuyoung Kim
Modified: 2019-05-02 16:22 PDT (History)
9 users (show)

See Also:


Attachments
Patch (1.15 KB, patch)
2016-06-27 21:42 PDT, Gyuyoung Kim
no flags Details | Formatted Diff | Diff
Patch (1.41 KB, patch)
2016-06-27 22:23 PDT, Gyuyoung Kim
no flags Details | Formatted Diff | Diff
Patch (1.19 KB, patch)
2016-06-28 00:33 PDT, Gyuyoung Kim
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gyuyoung Kim 2016-06-27 21:41:38 PDT
r202502 made a build warning because there is no default return value in TypeMap::at().

[107/4159] Building CXX object Source/JavaScriptCore/CMakeFiles/JavaScriptCore.dir/b3/B3InsertionSet.cpp.o
In file included from ../../Source/JavaScriptCore/b3/B3InsertionSet.h:33:0,
                 from ../../Source/JavaScriptCore/b3/B3InsertionSet.cpp:27:
../../Source/JavaScriptCore/b3/B3TypeMap.h: In member function ‘T& JSC::B3::TypeMap<T>::at(JSC::B3::Type) [with T = JSC::B3::Value*]’:
../../Source/JavaScriptCore/b3/B3TypeMap.h:62:5: warning: control reaches end of non-void function [-Wreturn-type]
     }
     ^
Comment 1 Gyuyoung Kim 2016-06-27 21:42:43 PDT
Created attachment 282210 [details]
Patch
Comment 2 Yusuke Suzuki 2016-06-27 22:02:03 PDT
I think we should fix this like that https://bugs.webkit.org/show_bug.cgi?id=152887.
See the context. https://bugs.webkit.org/show_bug.cgi?id=150794#c7
Comment 3 Gyuyoung Kim 2016-06-27 22:23:56 PDT
Created attachment 282212 [details]
Patch
Comment 4 Gyuyoung Kim 2016-06-27 22:24:35 PDT
(In reply to comment #2)
> I think we should fix this like that
> https://bugs.webkit.org/show_bug.cgi?id=152887.
> See the context. https://bugs.webkit.org/show_bug.cgi?id=150794#c7

Looks like this fix is better than me. Fixed.
Comment 5 Yusuke Suzuki 2016-06-27 22:26:33 PDT
Comment on attachment 282212 [details]
Patch

r=me
Comment 6 Gyuyoung Kim 2016-06-28 00:33:10 PDT
Created attachment 282219 [details]
Patch
Comment 7 Gyuyoung Kim 2016-06-28 00:38:19 PDT

*** This bug has been marked as a duplicate of bug 159194 ***