Bug 33037 - [check-webkit-style] Include order should understand uppercase
Summary: [check-webkit-style] Include order should understand uppercase
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Nobody
Depends on:
Reported: 2009-12-29 18:35 PST by Adam Barth
Modified: 2009-12-30 17:34 PST (History)
1 user (show)

See Also:

Patch (2.71 KB, patch)
2009-12-29 18:36 PST, Adam Barth
abarth: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Barth 2009-12-29 18:35:02 PST
[check-webkit-style] Include order should understand uppercase
Comment 1 Adam Barth 2009-12-29 18:36:06 PST
Created attachment 45627 [details]
Comment 2 WebKit Review Bot 2009-12-29 18:39:15 PST
style-queue ran check-webkit-style on attachment 45627 [details] without any errors.
Comment 3 Eric Seidel (no email) 2009-12-29 23:52:55 PST
Comment on attachment 45627 [details]

I'm slightly confused by this.  I think the sort order sorta defined to be whatever XCode's "sort lines" applescript thing does.  This makes our sort order looser, which is probably good, but I'm not sure which editors will care.
Comment 4 Adam Barth 2009-12-30 14:02:04 PST
Oh, we could do that.  The current thing is very braindead.  It thinks that capital letter always go ahead (or behind) their lowercase compatriots.


So, it wants things sorted as above, which is nonsensical to me.
Comment 5 Mark Rowe (bdash) 2009-12-30 16:42:07 PST
Our style guide says:

> Other #include statements should be in sorted order (case sensitive, as done by the command-line sort tool or the Xcode sort selection command).

This patch appears to be trying to enforce case-insensitive ordering.
Comment 6 Adam Barth 2009-12-30 17:32:41 PST
Comment on attachment 45627 [details]

Ah, so this patch is wrong then.  I think the current behavior is wrong too, but I'll check against the "sort" command.
Comment 7 Adam Barth 2009-12-30 17:34:34 PST
According to "sort", the script is currently correct.  Wacky, but correct.

$ echo "aa
> Ab" | sort