On 5 May 2009, at 11:46 AM, Luc Bourhis wrote: > > On 5 May 2009, at 11:19, Christiaan Hofman wrote: > >>> On the contrary, my code >>> version system (git for info) is set up to refuse committing >>> anything with trailing white spaces. Actually, please read on… >> >> Why on earth would you want to do that? > > This is a very common practice whose rationale is that white spaces > are a distraction in diffs. By forbidding them, only those lines > with meaningful differences are displayed in diffs. >
Maybe this may make sense when you're managing files you write yourself. Otherwise, IMHO it makes no sense to do this. >> Yes, that's what Adam said. And it's not a bug. > > Let's say it is an unwanted feature. Frankly, removing those spaces > will take you 2s, won't harm anybody who is not concerned with code > versioning and it will help those who are. It would seem to me that > it is a no-brainer of a patch. > 1. I don't see a good reason to have the white space at the end, ut it's there 2. I don't see a good reason to forbid trailing whitespace 3. There is good reason not to change anything in the code, because it will generate unwanted diffs So in fact it /does/ harm when I remove it (especially with those concerned with code versioning). Therefore I won't. Christiaan > Not a big deal anyway now that I have checked out the source, since > I fixed the code and it does what I want now! > > Luc Bourhis > Computer Scientist > Chemical Crystallography Laboratory > University of Durham, UK ------------------------------------------------------------------------------ The NEW KODAK i700 Series Scanners deliver under ANY circumstances! Your production scanning environment may not be a perfect world - but thanks to Kodak, there's a perfect scanner to get the job done! With the NEW KODAK i700 Series Scanner you'll get full speed at 300 dpi even with all image processing features enabled. http://p.sf.net/sfu/kodak-com _______________________________________________ Bibdesk-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/bibdesk-users
