RESOLVED DUPLICATE of bug 29317 19963
"$OpenBSD$" markers display in PrettyPatch, but not built-in "diff" 
https://bugs.webkit.org/show_bug.cgi?id=19963
Summary "$OpenBSD$" markers display in PrettyPatch, but not built-in "diff"
David Kilzer (:ddkilzer)
Reported 2008-07-09 11:17:17 PDT
* SUMMARY An OpenBSD patch on Bug 19775 uses "$OpenBSD$" as a marker between file diffs. The built-in "diff" format on bugs.webkit.org does not display these lines: https://bugs.webkit.org/attachment.cgi?id=22185&action=diff However, PrettyPatch does display them (after skipping the first) at the end of each diff: https://bugs.webkit.org/attachment.cgi?id=22185&action=prettypatch I think counting the number of actual/expected lines in the patch while parsing out individual diffs may fix the issue. (Would be interesting to look at how the "diff" format does its parsing, too.)
Attachments
Add attachment proposed patch, testcase, etc.
David Kilzer (:ddkilzer)
Comment 1 2008-07-10 15:07:54 PDT
Another example: https://bugs.webkit.org/attachment.cgi?id=22203&action=prettypatch
Caio Marcelo de Oliveira Filho
Comment 2 2011-03-08 17:50:26 PST
David Kilzer, I think my patch for bug 29317 will solve this problem as well.
Caio Marcelo de Oliveira Filho
Comment 3 2011-03-10 20:17:32 PST
This is fixed now. I'm marking as duplicate of bug 29317 because it is fundamentally the same problem: displaying lines that are not part of the actual diff contents. *** This bug has been marked as a duplicate of bug 29317 ***
David Kilzer (:ddkilzer)
Comment 4 2011-03-15 10:51:11 PDT
(In reply to comment #2) > David Kilzer, I think my patch for bug 29317 will solve this problem as well. Looks good. Thanks!
Note You need to log in before you can comment on or make changes to this bug.