From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 28916 invoked by alias); 21 Jun 2005 01:41:35 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 28884 invoked by uid 22791); 21 Jun 2005 01:41:29 -0000 Received: from lakermmtao10.cox.net (HELO lakermmtao10.cox.net) (68.230.240.29) by sourceware.org (qpsmtpd/0.30-dev) with ESMTP; Tue, 21 Jun 2005 01:41:29 +0000 Received: from white ([68.9.64.121]) by lakermmtao10.cox.net (InterMail vM.6.01.04.00 201-2131-118-20041027) with ESMTP id <20050621014127.XEUK7787.lakermmtao10.cox.net@white>; Mon, 20 Jun 2005 21:41:27 -0400 Received: from bob by white with local (Exim 3.35 #1 (Debian)) id 1DkXlX-0000h6-00; Mon, 20 Jun 2005 21:41:27 -0400 Date: Tue, 21 Jun 2005 01:41:00 -0000 From: Bob Rossi To: Nick Roberts Cc: gdb-patches@sources.redhat.com Subject: Re: [PATCH] Removal of markup annotations Message-ID: <20050621014127.GA2648@white> Mail-Followup-To: Nick Roberts , gdb-patches@sources.redhat.com References: <17071.40307.949193.158796@farnswood.snap.net.nz> <20050615154057.GB20778@white> <17072.46112.23475.26195@farnswood.snap.net.nz> <20050615233529.GC21803@white> <17072.49436.811019.408854@farnswood.snap.net.nz> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <17072.49436.811019.408854@farnswood.snap.net.nz> User-Agent: Mutt/1.3.28i X-SW-Source: 2005-06/txt/msg00324.txt.bz2 On Thu, Jun 16, 2005 at 12:00:28PM +1200, Nick Roberts wrote: > > > > Out of all of the annotations that you are removing here, CGDB only > > > > depends on annotate_breakpoints_headers, annotate_field, > > > > annotate_breakpoints_table, annotate_record and > > > > annotate_breakpoints_table_end. > > > > > > These are undocumented annotations and just mark up the output. Could > > > you parse it from the syntax instead? > > > > These were documented when I wrote the CGDB annotation subsystem. > > Between now and then, someone changed the documentation. > > > > This link may work to demonstrate the fact. > > http://web.archive.org/web/20030627071226/sources.redhat.com/gdb/current/onlinedocs/gdb_toc.html > > OK but they still just mark up the output. Well, it is not as easy for me to just parse the output without the annotations. Since I am still going to use annotate=2 (we are not depricating the whole thing, right?), then if you remove the breakpoints markup I'll have to handle old GDB's that have the markup, and new GDB's that don't. This would be a serious pain. > > I could look into parsing the syntax, but I would prefered if these > > stayed. Is this not desirable for some reason? > > They make the code in breakpoint.c harder to read. It's a question of > balancing the inconvenience of keeping them with that of removing them. I really only need 3 annotations out of the ones you are removing. breakpoint-table (to set a state that breakpoints are coming), field 5 (which we could rename to breakpoint-at), and breakpoint-table-end (to set a state that the breakpoints are over. To make things even easier, there could simply be a -breakpoint-begin, and a -breakpoint-end annotation, and those two annotations could mark up around the breakpoint. Would this clean up the code in breakpoint.c at all? Thanks, Bob Rossi