Summary: | [Win] Use of 1-bit Enum type behaves improperly | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brent Fulgham <bfulgham> | ||||
Component: | JavaScriptCore | Assignee: | Brent Fulgham <bfulgham> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bfulgham, mark.lam, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Brent Fulgham
2014-10-06 17:22:31 PDT
See also Bug 134252. Created attachment 239369 [details]
Patch
Comment on attachment 239369 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=239369&action=review r=me with the comment added. > Source/JavaScriptCore/debugger/Debugger.h:195 > + unsigned m_steppingMode : 1; I recall Darin saying that the accepted practice is to add a trailing comment to indicate the expected type: unsigned m_steppingMode : 1; // SteppingMode Comment on attachment 239369 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=239369&action=review >> Source/JavaScriptCore/debugger/Debugger.h:195 >> + unsigned m_steppingMode : 1; > > I recall Darin saying that the accepted practice is to add a trailing comment to indicate the expected type: > > unsigned m_steppingMode : 1; // SteppingMode Will do! Committed in r174381. <http://trac.webkit.org/changeset/174381> |