Bug 37088

Summary: Fix some "explicit braces to avoid ambiguous 'else'" warnings
Product: WebKit Reporter: Antonio Gomes <tonikitoo>
Component: Tools / TestsAssignee: Antonio Gomes <tonikitoo>
Severity: Normal CC: laszlo.gombos
Priority: P3    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Description Flags
(committed in r57077) patch none

Description Antonio Gomes 2010-04-05 08:42:52 PDT
it'd be good shut up some compiler warnings.

./../../../WebCore/dom/Node.cpp: In member function 'bool WebCore::Node::dispatchGenericEvent(WTF::PassRefPtr<WebCore::Event>)':
../../../../WebCore/dom/Node.cpp:2614: warning: suggest explicit braces to avoid ambiguous 'else'
uninitialized in this function

../../../../WebCore/page/DOMWindow.cpp: In member function 'bool WebCore::DOMWindow::dispatchEvent(WTF::PassRefPtr<WebCore::Event>, WTF::PassRefPtr<WebCore::EventTarget>)':
../../../../WebCore/page/DOMWindow.cpp:1437: warning: suggest explicit braces to avoid ambiguous 'else'
../../../../WebCore/platform/UUID.cpp: In function 'WebCore::String WebCore::createCanonicalUUIDString()':
Comment 1 Antonio Gomes 2010-04-05 08:45:12 PDT
Created attachment 52536 [details]
(committed in r57077) patch
Comment 2 Darin Adler 2010-04-05 08:45:51 PDT
Comment on attachment 52536 [details]
(committed in r57077) patch

These are braces, not parentheses. I'm happy that the compiler warnings happen to match our coding style.
Comment 3 Antonio Gomes 2010-04-05 08:48:38 PDT
thx darin! i will fix ChangeLog before landing