RESOLVED FIXED 52834
handle cases where PrettyPatch.rb doesn't linkify filenames 
https://bugs.webkit.org/show_bug.cgi?id=52834
Summary handle cases where PrettyPatch.rb doesn't linkify filenames
Ojan Vafai
Reported 2011-01-20 12:46:43 PST
handle cases where PrettyPatch.rb doesn't linkify filenames
Attachments
Patch (2.57 KB, patch)
2011-01-20 12:47 PST, Ojan Vafai 		abarth: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ojan Vafai
Comment 1 2011-01-20 12:47:06 PST
Created attachment 79638 [details] Patch
Eric Seidel (no email)
Comment 2 2011-01-20 13:39:13 PST
Comment on attachment 79638 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=79638&action=review I think this is really an aroben patch to review. > Websites/bugs.webkit.org/code-review.js:343 > + file_link.target = "_blank"; > + file_link.href += url_hash; > + diff_links.append(tracLinks(file_name, url_hash)); two space indent?
Ojan Vafai
Comment 3 2011-01-20 13:43:12 PST
Comment on attachment 79638 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=79638&action=review >> Websites/bugs.webkit.org/code-review.js:343 >> + diff_links.append(tracLinks(file_name, url_hash)); > > two space indent? Following the style of the file. Eventually we should make this file follow something closer to standard WebKit style.
Adam Barth
Comment 4 2011-01-20 13:51:52 PST
jQuery in four-space indent is painful, which is why the file uses two-space indent.
Ojan Vafai
Comment 5 2011-01-20 16:05:03 PST
Committed r76302: <http://trac.webkit.org/changeset/76302>
Note You need to log in before you can comment on or make changes to this bug.