From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Dan Nicolaescu Newsgroups: gmane.emacs.devel Subject: Re: new VC, current with changes up to 1.456 Date: Thu, 04 Oct 2007 11:57:11 -0700 Message-ID: <200710041857.l94IvBrN007742@oogie-boogie.ics.uci.edu> References: <20071003192456.GA8963@thyrsus.com> <200710040110.l941ASto012574@oogie-boogie.ics.uci.edu> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: sea.gmane.org 1191524295 30207 80.91.229.12 (4 Oct 2007 18:58:15 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Thu, 4 Oct 2007 18:58:15 +0000 (UTC) Cc: esr@thyrsus.com, emacs-devel@gnu.org To: rms@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Thu Oct 04 20:58:12 2007 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by lo.gmane.org with esmtp (Exim 4.50) id 1IdVtj-00061f-U7 for ged-emacs-devel@m.gmane.org; Thu, 04 Oct 2007 20:58:12 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1IdVtf-0001fH-SU for ged-emacs-devel@m.gmane.org; Thu, 04 Oct 2007 14:58:07 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1IdVtc-0001ey-L7 for emacs-devel@gnu.org; Thu, 04 Oct 2007 14:58:04 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1IdVtb-0001ef-7M for emacs-devel@gnu.org; Thu, 04 Oct 2007 14:58:03 -0400 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1IdVtb-0001ec-27 for emacs-devel@gnu.org; Thu, 04 Oct 2007 14:58:03 -0400 Original-Received: from oogie-boogie.ics.uci.edu ([128.195.1.41]) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1IdVtU-0005hN-7J; Thu, 04 Oct 2007 14:57:56 -0400 Original-Received: from mothra.ics.uci.edu (mothra.ics.uci.edu [128.195.6.93]) by oogie-boogie.ics.uci.edu (8.13.6/8.13.6) with ESMTP id l94IvBrN007742; Thu, 4 Oct 2007 11:57:11 -0700 (PDT) In-Reply-To: (Richard Stallman's message of "Thu\, 04 Oct 2007 13\:27\:32 -0400") Original-Lines: 20 X-ICS-MailScanner: Found to be clean X-ICS-MailScanner-SpamCheck: not spam, SpamAssassin (score=-0.84, required 5, autolearn=disabled, ALL_TRUSTED -1.44, J_CHICKENPOX_22 0.60) X-ICS-MailScanner-From: dann@mothra.ics.uci.edu X-Detected-Kernel: Solaris 9 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:80247 Archived-At: Richard Stallman writes: > It makes a huge > difference when looking at the files with ediff. I agree that your new > ordering makes more sense, but it makes it much harder to look only > for the things that actually changed (most of the functions move have > not changed at all...). > > The clean way to do this is to install two separate changes: one with > only substantial changes, not altering the order of the functions, and > the other changing ONLY the order of the functions and making no real > change in the code. > > How about if you (or someone) reorders the existing functions to match > the new version? Check that in, and it will be easy to compare the > new version against the current one. In the interest of getting this thing merged, I can reorder the vc.el functions in CVS (if nobody beats me to it). I can't get to it until tonight (PDT).