WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 95779
Fix style for Event.h
https://bugs.webkit.org/show_bug.cgi?id=95779
Summary
Fix style for Event.h
nbhargava
Reported
2012-09-04 13:44:58 PDT
Fix style for Event.h
Attachments
Patch
(11.48 KB, patch)
2012-09-04 13:46 PDT
,
nbhargava
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
nbhargava
Comment 1
2012-09-04 13:46:14 PDT
Created
attachment 162096
[details]
Patch
Robert Hogan
Comment 2
2012-09-04 14:03:29 PDT
This seems like a bad idea to me - we only fix style issues as we go rather than wholesale.
Eric Seidel (no email)
Comment 3
2012-09-04 14:04:33 PDT
Comment on
attachment 162096
[details]
Patch LGTM.
Eric Seidel (no email)
Comment 4
2012-09-04 14:04:55 PDT
Any time you touch this file it will complain. someone has to re-indent it eventually. :)
nbhargava
Comment 5
2012-09-04 14:14:12 PDT
Yeah, I only changed this one because another set of changes I made touched this file. I figured it would be less messy/easier to understand by just breaking them into two separate commits.
WebKit Review Bot
Comment 6
2012-09-04 15:22:18 PDT
Comment on
attachment 162096
[details]
Patch Clearing flags on attachment: 162096 Committed
r127514
: <
http://trac.webkit.org/changeset/127514
>
WebKit Review Bot
Comment 7
2012-09-04 15:22:21 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug