From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5451 invoked by alias); 21 Jul 2008 16:54:25 -0000 Received: (qmail 5439 invoked by uid 22791); 21 Jul 2008 16:54:24 -0000 X-Spam-Check-By: sourceware.org Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.31) with ESMTP; Mon, 21 Jul 2008 16:53:59 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id E329F2A972D; Mon, 21 Jul 2008 12:53:57 -0400 (EDT) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id wd8OUoxW2h3g; Mon, 21 Jul 2008 12:53:57 -0400 (EDT) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id A1D622A9728; Mon, 21 Jul 2008 12:53:57 -0400 (EDT) Received: by joel.gnat.com (Postfix, from userid 1000) id BA50EE7ACD; Mon, 21 Jul 2008 09:53:55 -0700 (PDT) Date: Mon, 21 Jul 2008 16:54:00 -0000 From: Joel Brobecker To: Tom Tromey Cc: vladimir@codesourcery.com, gdb-patches@sources.redhat.com Subject: GDB/MI PING (was: "Re: RFA: Remove gdb-events") Message-ID: <20080721165355.GA3837@adacore.com> References: <20080714171004.GC3998@adacore.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.4.2.2i Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2008-07/txt/msg00394.txt.bz2 Hi Vladimir, Tom made some changes in MI source files. They should not affect MI behavior at all, and they look fine to me, but I thought you might want to have a look. http://www.sourceware.org/ml/gdb-patches/2008-07/msg00244.html Hey Tom, > Joel> Eli: if you could take a look at the documentation of the new > Joel> observers, that would be great! I'm going to approve the > Joel> technical part, and we'll follow any suggestion you might have > Joel> either before or after commit. > > Joel> Vladimir: This patch touches MI. Could you take a look at this part > Joel> of the patch? The changes seem fine to me, but see below my suggestion. > > I'm waiting for these two, so I haven't sent an updated patch. > But... I see Eli answered, so let's now wait for Vladimir - I think he just missed this email (so I changed the subject to contain "MI" to catch his attention). > I checked and all the dependencies are there, afaict. I think you > just misread the tracepoint.o hunk. See the second insertion: Argh, you are right. Sorry about that. You should commit your change as soon as Vladimir OKs the MI part. I'm almost tempted to say that you should commit them now, and fix any comment that Vladimir might have as a followup patch, but Vladimir is usually extremely responsive. -- Joel