check-webkit-style returns non-zero when patch is entirely minus lines Total errors found: 0 in 0 files This is caused by PatchChecker.check which ignores non-add/modify lines. It should at least notice the lines and count the file, even if it doesn't say there are errors there.
When this is addressed, we also need to make sure that files are never double-counted, e.g. in the case of an SVN file move, which shows up in the diff as a copy followed by a delete.
*** Bug 39690 has been marked as a duplicate of this bug. ***
Hi, I think the number of files should leave as it stands because it should represent the actual number of files that are checked. So I'd like to fix this issue by counting files that contains only deleted lines separately. I'll send patch soon. Thanks,
Created attachment 63427 [details] Proposed Patch
Comment on attachment 63427 [details] Proposed Patch We need to update unittest for them. WebKitTools/Scripts/webkitpy/style/filereader.py:61 + self.processed_file_count = 0 Please update the docstring above.
(In reply to comment #5) Hamaji-san, Thank you for your quick review. > (From update of attachment 63427 [details]) > We need to update unittest for them. I'm sorry for missing tests. I'll add unittests for them. > WebKitTools/Scripts/webkitpy/style/filereader.py:61 > + self.processed_file_count = 0 > Please update the docstring above. I've noticed that the variable also counts skipped files so that I'll revert the change.
> > (From update of attachment 63427 [details] [details]) > > We need to update unittest for them. > > I'm sorry for missing tests. I'll add unittests for them. Never mind. Thanks for fixing this issue!
Created attachment 63431 [details] Proposed Patch V2
Comment on attachment 63431 [details] Proposed Patch V2 Looks good.
Comment on attachment 63431 [details] Proposed Patch V2 Clearing flags on attachment: 63431 Committed r64743: <http://trac.webkit.org/changeset/64743>
All reviewed patches have been landed. Closing bug.