Summary: | “Formatted Diff” view mangles non-ASCII characters | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | mitz | ||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED DUPLICATE | ||||||
Severity: | Normal | CC: | ap, aroben, bdakin, darin, dbates, ddkilzer, haraken, mmaxfield, mrobinson, pknight, tkent | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
URL: | https://bugs.webkit.org/attachment.cgi?id=120812&action=prettypatch | ||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=238630 | ||||||
Attachments: |
|
Description
mitz
2011-12-30 12:21:35 PST
Bugzilla.pm: > sub init_page { > (binmode STDOUT, ':utf8') if Bugzilla->params->{'utf8'}; attachment.cgi sub prettyPatch: > open2(\*OUT, \*IN, "/usr/bin/ruby", "-I", "PrettyPatch", "PrettyPatch/prettify.rb", "--html-exceptions"); > $ENV{'PATH'} = $orig_path; > print IN $attachment->data; > close(IN); > while (<OUT>) { > print; > } > close(OUT); I guess OUT works as binary, and "print" converts a binary line to UTF-8 because of binmode. Probably utf8::decode($_); before print fixes this? *** This bug has been marked as a duplicate of bug 45760 *** This is not fixed. The apostrophes look correct to me now. Attaching screenshot. Created attachment 456319 [details]
Screenshot
(In reply to Myles C. Maxfield from comment #4) > The apostrophes look correct to me now. Attaching screenshot. Good to see! Do you think this was fixed along with bug 238630? Yes. *** This bug has been marked as a duplicate of bug 238630 *** |