Summary: | [chromium] WebURL should have a < operator | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jay Civelli <jcivelli> | ||||||||
Component: | WebKit Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | abarth, commit-queue, fishd, levin | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | PC | ||||||||||
OS: | OS X 10.5 | ||||||||||
Attachments: |
|
Description
Jay Civelli
2010-06-15 11:54:04 PDT
Created attachment 58803 [details]
Initial patch
Comment on attachment 58803 [details] Initial patch Rejecting patch 58803 from commit-queue. Failed to run "[u'git', u'svn', u'dcommit']" exit_code: 1 Last 500 characters of output: geLog M WebKit/chromium/public/WebCString.h M WebKit/chromium/public/WebURL.h M WebKit/chromium/src/WebCString.cpp A repository hook failed: MERGE request failed on '/repository/webkit/trunk': Commit blocked by pre-commit hook (exit code 1) with output: svnlook: Can't write to stream: Broken pipe The following ChangeLog files contain OOPS: trunk/WebKit/chromium/ChangeLog Please don't ever say "OOPS" in a ChangeLog file. at /usr/local/git/libexec/git-core/git-svn line 570 Full output: http://webkit-commit-queue.appspot.com/results/3273301 Created attachment 59130 [details]
Fixed ChangeLog
Created attachment 59161 [details]
Fixed bug: comparing "a" and "aa" would return strings as equal.
Changed the code to use strcmp instead of strncmp since CString are NULL terminated.
No fishd review for WebKit API change? And I'm blind. Sorry for the noise. *** Bug 40861 has been marked as a duplicate of this bug. *** Comment on attachment 59161 [details] Fixed bug: comparing "a" and "aa" would return strings as equal. Clearing flags on attachment: 59161 Committed r61495: <http://trac.webkit.org/changeset/61495> All reviewed patches have been landed. Closing bug. |