Summary: | check-webkit-style: Rename CppProcessor to CppChecker, etc | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Jerdonek <cjerdonek> | ||||
Component: | Tools / Tests | Assignee: | Chris Jerdonek <cjerdonek> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | cjerdonek, commit-queue, eric, hamaji, levin | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Chris Jerdonek
2010-04-28 07:37:43 PDT
Note that this naming change was already started in-- https://bugs.webkit.org/show_bug.cgi?id=38122 (renamed processors/ folder to checkers/) https://bugs.webkit.org/show_bug.cgi?id=37066 (create new StyleProcessor class) Created attachment 54564 [details]
Proposed patch
No logic changes in this patch -- only renames (many using grep).
Can anyone take a look at this? There really are no changes aside from renames. Thanks. Comment on attachment 54564 [details]
Proposed patch
I'm not sure I really understand the distinction, so all you really get is a rubber-stamp. Why changing all this to checker, instead of the other way around? Seems processor is about as meaningless as checker so I would probably optimize for minimal renames.
But honestly, I have really no opinions about any of this.
Do we have any need to be able to merge any of this code with anyone?
(In reply to comment #4) Thanks, Eric. > (From update of attachment 54564 [details]) > I'm not sure I really understand the distinction, so all you really get is a > rubber-stamp. Why changing all this to checker, instead of the other way > around? Seems processor is about as meaningless as checker so I would probably > optimize for minimal renames. In a nutshell, check-webkit-style uses a ProcessorBase class which can be used for any operations on files (e.g. operations other than "checking," like extracting license texts for bug 35465). CppProcessor, PythonProcessor, etc. do not inherit from ProcessorBase, so I wanted to choose a more specific name. Since those classes are limited to checking style, *Checker made sense. > Do we have any need to be able to merge any of this code with anyone? Not to my knowledge. Thanks for the update. (In reply to comment #5) > > I'm not sure I really understand the distinction, so all you really get is a > > rubber-stamp. Why changing all this to checker, instead of the other way > > around? Seems processor is about as meaningless as checker so I would probably > > optimize for minimal renames. > > In a nutshell, check-webkit-style uses a ProcessorBase class which can be used > for any operations on files (e.g. operations other than "checking," like > extracting license texts for bug 35465). > > CppProcessor, PythonProcessor, etc. do not inherit from ProcessorBase, so I > wanted to choose a more specific name. Since those classes are limited to > checking style, *Checker made sense. A final bit of information: The StyleProcessor class is the class that check-webkit-style uses that does inherit from ProcessorBase. Internally it does various things including choosing which *Checkers to construct (CarriageReturnChecker, PythonChecker, etc) and calling those. Comment on attachment 54564 [details] Proposed patch Clearing flags on attachment: 54564 Committed r58742: <http://trac.webkit.org/changeset/58742> All reviewed patches have been landed. Closing bug. |