From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 31460 invoked by alias); 20 Nov 2006 02:37:01 -0000 Received: (qmail 31452 invoked by uid 22791); 20 Nov 2006 02:37:00 -0000 X-Spam-Check-By: sourceware.org Received: from viper.snap.net.nz (HELO viper.snap.net.nz) (202.37.101.8) by sourceware.org (qpsmtpd/0.31) with ESMTP; Mon, 20 Nov 2006 02:36:54 +0000 Received: from kahikatea.snap.net.nz (p202-124-125-20.snap.net.nz [202.124.125.20]) by viper.snap.net.nz (Postfix) with ESMTP id 9BA893D860D; Mon, 20 Nov 2006 15:37:18 +1300 (NZDT) Received: by kahikatea.snap.net.nz (Postfix, from userid 500) id 40F4ABE447; Mon, 20 Nov 2006 15:32:43 +1300 (NZDT) From: Nick Roberts MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-ID: <17761.5066.494599.865067@kahikatea.snap.net.nz> Date: Mon, 20 Nov 2006 02:37:00 -0000 To: Daniel Jacobowitz Cc: gdb-patches@sources.redhat.com Subject: Re: [PATCH] PR mi/2077 "set edit off" breaks MI In-Reply-To: <20061117213449.GC13961@nevyn.them.org> References: <17719.15772.325330.48627@kahikatea.snap.net.nz> <20061117213449.GC13961@nevyn.them.org> X-Mailer: VM 7.19 under Emacs 22.0.90.14 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2006-11/txt/msg00217.txt.bz2 > > This patch is taken out of the async patch. It's probably a good idea to > > commit the different parts separately anyway. > > Definitely. I can split it into one async patch (the largest part) and several small MI patches. The MI patches are probably quite safe changes and the async part shouldn't interfere with normal use. Unless Elena becomes active again, I think it's likely that you will be the only person who would want to/be willing to review the latter. You have suggested that you would also like to use it for other reasons. When you have time to review it please tell me and I will re-submit it (without the MI changes). > > Apple switch interpreters which fixes this bug. Jim Ingham can probably > > explain more eloquently why it works. > > FYI, the reason I shelved this one to reply to later was that you > didn't explain how it works. It's much harder to review a patch for > correctness in that case. I have to go figure it out myself > whether it's fixing the symptom or the cause. > > When I went to do that I decided that the patch only fixed the symptom, > unfortunately. Try repeating your test case, but instead of saying > "set edit off", say "-gdb-set edit off". That won't switch > interpreters because it isn't running a CLI command. Then try > -exec-next and it will fail. Ah, I see. I'll look into it. >... > > Otherwise can I just write?: > > > > 2006-10-19 Apple Computer, Inc > > > > Or do I need a person/e-mail address? > > I think it would be best to put your own name on the changelog, and > mention something like "From Apple Computer, Inc." at the top of the > entry. I don't really know though so if anyone else has an opinion, > please share. I'll ask RMS, I've always been told that the committer's name should not be the one to go in the changelog. -- Nick http://www.inet.net.nz/~nickrob