epriestley
9844bbb4f9
Summary: Diffs with missing context don't render properly in the "ignore all whitespace" algorith, so don't try to use it. These diffs can occur if someone creates a diff via the web interface, for example, or if they muck around in their copy of 'arc'. See D473, T246 (a problem with D473), rPe5bb756b5191720 (revert of D473) and T231. Test Plan: Viewed a diff with missing context from the web interface. Verified normal diffs still rendered with all whitespace ignored. Reviewed By: fratrik Reviewers: jungejason, aran, tuomaspelkonen, fratrik Commenters: jungejason CC: aran, epriestley, fratrik, jungejason Differential Revision: 500
PROJECT STATUS: CAVEAT EMPTOR This is an unstable preview release. You can learn more at http://phabricator.org/ as well as click around our development install. Developer mailing list at https://groups.google.com/group/phabricator-dev and please report issues using GitHub. WHAT IS PHABRICATOR? Phabricator is a suite of web applications that facilitate software development tasks, particularly code review. The primary application in the suite is Differential, a code review tool. Phabricator is highly unstable and has many missing features! These applications are being brought over from Facebook's internal toolset, but there's a lot of stuff that hasn't made it over yet. Feel free to follow the project but you probably shouldn't try to install this yet unless you're extremely ambitious or just want to take a look at it. LICENSE Phabricator is released under the Apache 2.0 license except as otherwise noted. http://www.apache.org/licenses/LICENSE-2.0