Jan Nijtmans wrote:
Therefore I reverted the "diff-eolws" branch diffFlags to what they
are in trunk now: DIFF_IGNORE_EOLWS.

Sounds good. Let's get this bug-fix-turned-feature branch into trunk! :)

Question to anyone interested:
       What should be the diffFlags used for blame/annotate
       1) DIFF_IGNORE_EOLWS   (the current behavior, ignoring all
end-of-line spacing)
       2) DIFF_IGNORE_SOLWS|DIFF_IGNORE_EOLWS (my preference, ignoring
            all start-of-line and end-of-line spacing)
       3) 0:  Any difference in spacing is considered different
       4) Any other flag, to-be-implemented.

My personal preference is for (1), as EOL whitespace is practically never worth caring about in any way (although, as Martin pointed out, that isn't always the case). I'd also guess that it makes blame/annotate more useful for the majority of users than (3), but maybe I'm wrong on that. I wouldn't argue against (3) as the default, though.
_______________________________________________
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users

Reply via email to