Tsuna <tsuna@warszawa.lrde.epita.fr> writes:
On 2006-11-03, Roland Levillain <roland@lrde.epita.fr> wrote:
Roland Levillain <roland@lrde.epita.fr> writes:
https://svn.lrde.epita.fr/svn/oln/trunk/metalic
Index: ChangeLog from Roland Levillain <roland@lrde.epita.fr>
Add spaces in macros to please icpc. * mlc/is_a.hh, mlc/case.hh: Add spaces in macros to please icpc.
0 files changed
Index: mlc/is_a.hh Index: mlc/case.hh
Sorry for this ``blank'' patch, but Vcs removes any white space in altered lines in diffs.
BTW, is there an option to disable this behavior in Vcs? If not, I suggest adding an option like `--show-space-change' (i.e., the opposite from diff's --ignore-space-change).
Or simply when the diff is empty, diff it again without removing white spaces and stuff.
The choice of displaying whitespace changes or not must be left to the user, even if there's a default behavior (not to show them, in the current case). And the problem is even bigger: think of Python files, where indentation matters. For this case, Vcs or svn-wrapper could use libmagic (i.e., `file') to check the type of the file(s), and adjust its behavior. P.S.: BTW, I suspect `file -i' of being buggy on Debian: brasilia ~ % file ~/src/tc/dev/ast-nodes-gen 19:12 #210 /home/levill_r/src/tc/dev/ast-nodes-gen: a python script text executable brasilia ~ % file -i ~/src/tc/dev/ast-nodes-gen 19:12 #211 /home/levill_r/src/tc/dev/ast-nodes-gen: text/x-java; charset=us-ascii :) (Micha just said to me he observed a similar behavior on Gentoo...)