Bug 138416

Summary: Fix the debug build after r175525
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: New BugsAssignee: Csaba Osztrogonác <ossy>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: commit-queue, darin, d-r, fmalita, gyuyoung.kim, krit, nikos.andronikos, ossy, pdr, schenney, sergio
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 137942    
Attachments:
Description Flags
Patch none

Description Csaba Osztrogonác 2014-11-05 03:43:53 PST
../../Source/WebCore/svg/SVGMarkerElement.cpp: In member function 'WebCore::SVGMarkerOrientType& WebCore::SVGMarkerElement::orientType() const':
../../Source/WebCore/svg/SVGMarkerElement.cpp:256:51: error: comparison of unsigned expression >= 0 is always true [-Werror=type-limits]
cc1plus: all warnings being treated as errors

line256: ASSERT(wrapper->currentAnimatedValue() >= 0 && wrapper->currentAnimatedValue() < SVGMarkerOrientMax);

wrapper->currentAnimatedValue() is unsigned, so it is always true, I think we should simple remove it.
Comment 1 Csaba Osztrogonác 2014-11-05 03:46:22 PST
Created attachment 241023 [details]
Patch
Comment 2 Csaba Osztrogonác 2014-11-05 04:09:50 PST

*** This bug has been marked as a duplicate of bug 138414 ***
Comment 3 Darin Adler 2014-11-05 09:04:26 PST

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