From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.io/gmane.emacs.gnus.general/64529 Path: news.gmane.org!not-for-mail From: Katsumi Yamaoka Newsgroups: gmane.emacs.gnus.general Subject: Re: Mixing whitespace and topical changes Date: Tue, 17 Apr 2007 11:04:38 +0900 Organization: Emacsen advocacy group Message-ID: References: NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: sea.gmane.org 1176775593 24985 80.91.229.12 (17 Apr 2007 02:06:33 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Tue, 17 Apr 2007 02:06:33 +0000 (UTC) To: ding@gnus.org Original-X-From: ding-owner+M13040@lists.math.uh.edu Tue Apr 17 04:06:27 2007 Return-path: Envelope-to: ding-account@gmane.org Original-Received: from util0.math.uh.edu ([129.7.128.18]) by lo.gmane.org with esmtp (Exim 4.50) id 1Hdd5O-00034D-Fx for ding-account@gmane.org; Tue, 17 Apr 2007 04:06:26 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.math.uh.edu) by util0.math.uh.edu with smtp (Exim 4.63) (envelope-from ) id 1Hdd4U-0000Ej-SU; Mon, 16 Apr 2007 21:05:30 -0500 Original-Received: from mx2.math.uh.edu ([129.7.128.33]) by util0.math.uh.edu with esmtps (TLSv1:AES256-SHA:256) (Exim 4.63) (envelope-from ) id 1Hdd4S-0000EH-PL for ding@lists.math.uh.edu; Mon, 16 Apr 2007 21:05:28 -0500 Original-Received: from quimby.gnus.org ([80.91.231.51]) by mx2.math.uh.edu with esmtp (Exim 4.63) (envelope-from ) id 1Hdd4M-0006vR-E4 for ding@lists.math.uh.edu; Mon, 16 Apr 2007 21:05:28 -0500 Original-Received: from orlando.hostforweb.net ([216.246.45.90]) by quimby.gnus.org with esmtp (Exim 3.35 #1 (Debian)) id 1Hdd4L-0008J4-00 for ; Tue, 17 Apr 2007 04:05:21 +0200 Original-Received: from [66.225.201.151] (port=47183 helo=mail.jpl.org) by orlando.hostforweb.net with esmtpa (Exim 4.63) (envelope-from ) id 1Hdd3r-0002jm-Fp for ding@gnus.org; Mon, 16 Apr 2007 21:04:52 -0500 X-Hashcash: 1:20:070417:ding@gnus.org::tuJHIeTDF2uR9Etm:00005xAM X-Face: #kKnN,xUnmKia.'[pp`;Omh}odZK)?7wQSl"4o04=EixTF+V[""w~iNbM9ZL+.b*_CxUmFk B#Fu[*?MZZH@IkN:!"\w%I_zt>[$nm7nQosZ<3eu;B:$Q_:p!',P.c0-_Cy[dz4oIpw0ESA^D*1Lw= L&i*6&( User-Agent: Gnus/5.110006 (No Gnus v0.6) Emacs/22.0.97 (gnu/linux) Cancel-Lock: sha1:MaQKbaCw19E1JD7GpBmtYM0yJa8= X-Antivirus-Scanner: Clean mail though you should still use an Antivirus X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - orlando.hostforweb.net X-AntiAbuse: Original Domain - gnus.org X-AntiAbuse: Originator/Caller UID/GID - [0 0] / [47 12] X-AntiAbuse: Sender Address Domain - jpl.org X-Source: X-Source-Args: X-Source-Dir: X-Spam-Score: -2.5 (--) List-ID: Precedence: bulk Xref: news.gmane.org gmane.emacs.gnus.general:64529 Archived-At: >>>>> In Didier Verna wrote: > Reiner Steib wrote: >> please avoid committing lots of whitespace changes mixed with topical >> changes. > Reiner, > I can understand your concern (I guess it mainly has to do with > synchronizing between the GNU Emacs version and the CVS version), but > I'm not ready to change my setup (all whitespace cleanup is done > automatically and I never see them in diffs) until this issue has been > officially resolved. I've tried to raise it several times in the past > but didn't get any interest from anybody. Note that I will accept > whatever decision the majority adopts, even if it goes against me :-) I did the same before, but now I keep clean only things that I changed. It is enough for me that Lars ever said to me "Don't use whitespace.el or equivalent when changing Gnus.". IIRC, he said he agrees to make source code clean all together but it should be done independently to topical changes. So, I don't mix whitespace changes and topical changes as much as possible. Or else, not only the maintainers who check the validity of changes but also people who read CVS diffs will be bothered if there are such changes. (Actually I was sometimes annoyed when I had been merging Gnus' changes into Semi-gnus in the past.) Making source code clean is generally good. But I think it should be done line by line manually at least in Gnus. What do you think about this? (insert "One space character is here> ") It is a bad habit of course, though. > Let me restate my position: I believe that the right thing to do is to > have whitespace-cleanup on write-file-hooks for everybody committing to > Gnus. This is simple to achieve, this effectively removes any such > problems and this makes the files [automatically] cleaner. There's also > the question of the info files on which I only got a "I've heard that it > is better this way" sort of message.[1] > Sorry to be a little pushy on this, but whitespace.el functions on a > major-mode basis and not on a file-name basis, so I would need some > trickery to make it stop working on Gnus files, so I really want a > discussion about this before. > Footnotes: > [1] FWIW, texinfo-mode is on whitespace-modes by default, and I'd never > heard any complaint about this until a few days ago. Let me write repeatedly; please don't use tabs in texi files. Especially for sections in which the indentation is meaningful, like @lisp ... @end lisp. Regards,