From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 25852 invoked by alias); 29 Jan 2013 19:45:33 -0000 Received: (qmail 25842 invoked by uid 22791); 29 Jan 2013 19:45:31 -0000 X-SWARE-Spam-Status: No, hits=-3.7 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,KHOP_SPAMHAUS_DROP,KHOP_THREADED,RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from cora.hrz.tu-chemnitz.de (HELO cora.hrz.tu-chemnitz.de) (134.109.228.40) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 29 Jan 2013 19:45:25 +0000 Received: from 91-66-49-158-dynip.superkabel.de ([91.66.49.158] helo=localhost) by cora.hrz.tu-chemnitz.de with esmtpsa (TLSv1:DHE-RSA-AES128-SHA:128) (Exim 4.80.1) (envelope-from ) id 1U0H7Q-0007j3-VF; Tue, 29 Jan 2013 20:45:21 +0100 Date: Tue, 29 Jan 2013 19:45:00 -0000 From: =?iso-8859-1?Q?Andr=E9_P=F6nitz?= To: Mircea Gherzan Cc: tromey@redhat.com, vladimir@codesourcery.com, marc.khouzam@ericsson.com, gdb-patches@sourceware.org Subject: Re: [RFC] Fix the MI result of -break-insert with multiple locations Message-ID: <20130129194520.GA3727@klara.mpi.htwm.de> References: <1359470164-32004-1-git-send-email-mircea.gherzan@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1359470164-32004-1-git-send-email-mircea.gherzan@intel.com> X-purgate: clean X-purgate-type: clean X-purgate-ID: 154106::1359488721-000004E7-E20C7B64/0-0/0-0 X-Scan-AV: cora.hrz.tu-chemnitz.de;2013-01-29 20:45:21;89fd588420520f579a6072e2b1d8be8b X-Scan-SA: cora.hrz.tu-chemnitz.de;2013-01-29 20:45:21;0300b60f1aeb41c3d235199559f8e696 X-Spam-Score: -1.0 (-) X-Spam-Report: --- Textanalyse SpamAssassin 3.3.1 (-1.0 Punkte) Fragen an/questions to: Postmaster TU Chemnitz * -1.0 ALL_TRUSTED Passed through trusted hosts only via SMTP --- Ende Textanalyse X-IsSubscribed: yes 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: 2013-01/txt/msg00693.txt.bz2 On Tue, Jan 29, 2013 at 03:36:04PM +0100, Mircea Gherzan wrote: > The current MI output when printing a breakpoint with multiple locations > is not conformant to the MI specification: > > bkpt={number="1", ...},{number="1.1", ...},{number="1.2", ...} > > This patch fixes this issue by moving the locations to a list inside the > first tuple: > > bkpt={number="1", ... , locations=[{number="1.1", ...}, ...]} This seems more like an additional burden for frontends which cannot rely on a specific gdb version being installed as they have to keep code to parse both results, for years. The fact that the current output might not conform to the documented grammar has no impact to existing frontends as they have/had to cope with this kind of slight deviations anyway. An alternative approach would be to just make the documentation match the actual output. This is not unprecedented. Andre'