Re: rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace)

Hi, Thien-Thi Nguyen writes: > () l...@gnu.org (Ludovic Courtès) > () Sat, 23 Jan 2010 17:10:50 +0100 > >To me it’s clear: status quo, i.e., no ‘whitespace-cleanup’ >or similar all around, but encourage people not to leave >trailing whitespace. > > OK, thanks for the clarification. >

Re: rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace)

() l...@gnu.org (Ludovic Courtès) () Sat, 23 Jan 2010 17:10:50 +0100 To me it’s clear: status quo, i.e., no ‘whitespace-cleanup’ or similar all around, but encourage people not to leave trailing whitespace. OK, thanks for the clarification. Please find attached a patch to HACKING. thi

Re: rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace)

Hi, Thien-Thi Nguyen writes: > Revised proposal: Let's decide one way or the other, To me it’s clear: status quo, i.e., no ‘whitespace-cleanup’ or similar all around, but encourage people not to leave trailing whitespace. Thanks, Ludo’.

Re: rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace)

() Neil Jerram () Thu, 21 Jan 2010 20:46:26 + > Jim Meyering gives a nice rationale in > . I'm afraid those rationales don't persuade me: [...] Well, i respect the work Jim Meyering does and read his rationale w/ a more gener

Re: rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace)

Thien-Thi Nguyen writes: > The above form lives in my Emacs init flow, causing trailing whitespace to be > deleted on `save-buffer' (C-x C-s). For many projects (but not Guile) this > DTRT, because trailing whitespace is not tolerated. Jim Meyering gives a > nice rationale in

Re: rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace)

Hi, Neil Jerram writes: > l...@gnu.org (Ludovic Courtès) writes: > >> (using ‘git blame’ becomes almost useless). > > Wouldn't `git blame -w' solve that problem? Hmm, possibly, yes (I didn’t know this option). Thanks, Ludo’.

Re: rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace)

l...@gnu.org (Ludovic Courtès) writes: > (using ‘git blame’ becomes almost useless). Wouldn't `git blame -w' solve that problem? Neil

Re: rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace)

Hi, Thien-Thi Nguyen writes: > Well, if there's concensus, i'm happy to do the janitorial duties. IMO the cons (blurring VCS history) outweighs the pros. But please (setq show-trailing-whitespace t). Thanks, Ludo’.

Re: rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace)

() Andy Wingo () Wed, 13 Jan 2010 21:02:03 +0100 I would be happier if my use of Emacs didn't leave me often with trailing whitespace. See subject line. But such as it is, I don't want to play manual garbage collector for stray spaces, which have no detriment on code. Well, if ther

Re: rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace)

On Wed 13 Jan 2010 10:06, Thien-Thi Nguyen writes: > I propose Guile also not tolerate trailing whitespace. What do people > think? To change all that code is a bit of a pain, for not very much gain. I personally don't care about the topic. I would be happier if my use of Emacs didn't leave me

Re: rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace)

() l...@gnu.org (Ludovic Courtès) () Wed, 13 Jan 2010 11:53:01 +0100 I agree, but adding such a hook would cause many gratuitous diffs, which then blur VCS history (using ‘git blame’ becomes almost useless). Usually, once a project decides to use some whitespace policy, there is a one-ti

Re: rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace)

Hi, Thien-Thi Nguyen writes: > The above form lives in my Emacs init flow, causing trailing whitespace to be > deleted on `save-buffer' (C-x C-s). For many projects (but not Guile) this > DTRT, because trailing whitespace is not tolerated. Jim Meyering gives a > nice rationale in