Looks good to me.
preserving line endings in diff files
Review Request #85 — Created June 19, 2007 and submitted
|Review Board SVN (deprecated)|
I am unlucky enough to have to deal with different line endings from different operation systems :) This patch will preserve line endings in diff files I see a few solutions, for various CRLF vs LF problems. This simple patch allows applying diffs to files generated on a CRLF OS, but where the diff was generated on a LF OS. I have not needed to try other combinations.
All unittests for diff parsing and patching pass