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+

Description Shivakumar J M 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) ?
Comment 1 Shivakumar J M 2014-11-05 03:23:04 PST
Created attachment 241020 [details]
Patch

Fixed debug build error.
Comment 2 Shivakumar J M 2014-11-05 03:26:54 PST
Issue might have occured by https://bugs.webkit.org/show_bug.cgi?id=137942
Comment 3 Csaba Osztrogonác 2014-11-05 04:09:50 PST
*** Bug 138416 has been marked as a duplicate of this bug. ***
Comment 4 Csaba Osztrogonác 2014-11-05 05:50:00 PST

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