"Vanderhoof, Tzadik" <tzadik.vanderh...@optum360.com> writes:
>> From: Duy Nguyen [mailto:pclo...@gmail.com] >> >> I face this "problem" every day, but most editors nowadays have block- >> based editing that would allow you to remove a column of "+/-" >> easily. At least it has not bothered me to think of improving it. > > Would a patch be welcome? I am not enthused for at least two reasons. The weaker one is "it would likely to introduce a lot of noise in the code for a feature of dubious merit". The other is a bit more serious. Cutting and pasting has been a source of lost or mangled whitespaces. Tabs get expanded, a wrapped long single line turns into two lines, an originally indented line auto-indented when inserted to the receiving editor, etc., etc., depending on the pager that the output was passed through for the terminal, the terminal program itself and the editor. The "feature" will encourage cut-and-paste, and I personally would be reluctant to add things that encourage bad practice to the users. As Duy said, saving the "diff" output to another file, opening in an editor that output file and the file the patch targets to modify, and transferring the lines while dropping unnecessray parts (i.e. unwanted context lines and preimage lines, and possibly undesired postimage lines, and also the leading SP/+/- designators) has no such downside. It obviously has an added benefit that it makes it less likely for people to cut and paste a line and then become unsure if they really cut from the green line or they by mistake also pasted an adjacent red line. So, I'd say the answer is "probably not".
