[Webkit-unassigned] [Bug 39514] check-webkit-style shouldn't complain about not including a primary header file if none exists.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jun 11 08:25:04 PDT 2010


Shinichiro Hamaji <hamaji at chromium.org> changed:

           What    |Removed                     |Added
  Attachment #57968|review?                     |review-
               Flag|                            |

--- Comment #5 from Shinichiro Hamaji <hamaji at chromium.org>  2010-06-11 08:25:03 PST ---
(From update of attachment 57968)
Yeah, I think we want some test cases for this change.

 +      def check_next_include_order(self, header_type, file_is_header, include_state):
I think we can just use self as include_state?

 +      for line_number in xrange(clean_lines.num_lines()):
It's unfortunate we need to iterate all lines twice. Cannot we implement this by one-pass approach? Maybe it should be like

1. If primary header comes before config.h, emit a warning when we find config.h
2. If primary header isn't found between config.h and "other headers", set has_primary_header=False
3. If primary header is found after we see at least one "other headers", emit a warning.

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