Bug 138414

Summary: Fix Debug Build Error in Webcore module
Product: WebKit Reporter: Shivakumar J M <shiva.jm>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: commit-queue, d-r, fmalita, gyuyoung.kim, krit, nikos.andronikos, ossy, pdr, schenney, sergio, zimmermann
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch darin: review+, darin: commit-queue+

Shivakumar J M
Reported 2014-11-05 03:14:25 PST
Getting below debug build error on latest Wk2-efl build. Wmissing-format-attribute -Wpointer-arith -Wundef -Wwrite-strings -MMD -MT Source/WebCore/CMakeFiles/WebCore.dir/svg/SVGMarkerElement.cpp.o -MF "Source/WebCore/CMakeFiles/WebCore.dir/svg/SVGMarkerElement.cpp.o.d" -o Source/WebCore/CMakeFiles/WebCore.dir/svg/SVGMarkerElement.cpp.o -c ../../Source/WebCore/svg/SVGMarkerElement.cpp In file included from ../../Source/WTF/wtf/PossiblyNull.h:29:0, from ../../Source/WTF/wtf/FastMalloc.h:26, from ../../Source/WebCore/config.h:75, from ../../Source/WebCore/svg/SVGMarkerElement.cpp:22: ../../Source/WebCore/svg/SVGMarkerElement.cpp: In member function ‘WebCore::SVGMarkerOrientType& WebCore::SVGMarkerElement::orientType() const’: ../../Source/WebCore/svg/SVGMarkerElement.cpp:256:55: error: comparison of unsigned expression >= 0 is always true [-Werror=type-limits] ASSERT(wrapper->currentAnimatedValue() >= 0 && wrapper->currentAnimatedValue() < SVGMarkerOrientMax); ^ ../../Source/WTF/wtf/Assertions.h:232:8: note: in definition of macro ‘ASSERT’ (!(assertion) ?
Attachments
Patch (1.46 KB, patch)
2014-11-05 03:23 PST, Shivakumar J M
darin: review+
darin: commit-queue+
Shivakumar J M
Comment 1 2014-11-05 03:23:04 PST
Created attachment 241020 [details] Patch Fixed debug build error.
Shivakumar J M
Comment 2 2014-11-05 03:26:54 PST
Csaba Osztrogonác
Comment 3 2014-11-05 04:09:50 PST
*** Bug 138416 has been marked as a duplicate of this bug. ***
Csaba Osztrogonác
Comment 4 2014-11-05 05:50:00 PST
*** This bug has been marked as a duplicate of bug 138407 ***
Note You need to log in before you can comment on or make changes to this bug.