[Webkit-unassigned] [Bug 57822] Build fix for YarrParser.h

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 5 14:59:55 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=57822





--- Comment #9 from Balazs Kelemen <kbalazs at webkit.org>  2011-04-05 14:59:54 PST ---
(In reply to comment #8)
> (From update of attachment 88191 [details])
> If we do make this change, it will be just to make the compiler happy. When m_state is Empty it is illegal to look at the character value, so initializing it is only helpful to mitigate programming mistakes. If m_state is set to another state where m_character is used, then m_character will be set appropriately.
> 
> The compiler can’t analyze the data flow well enough to see the relationship so it thinks there is a potential problem here, but there is none.

Sure. The only motivation behind this patch is to make the compiler happy.
Thanks for the review!

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list