From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8576 invoked by alias); 30 Jan 2008 04:25:15 -0000 Received: (qmail 8564 invoked by uid 22791); 30 Jan 2008 04:25:14 -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; Wed, 30 Jan 2008 04:24:47 +0000 Received: from kahikatea.snap.net.nz (192.31.255.123.static.snap.net.nz [123.255.31.192]) by viper.snap.net.nz (Postfix) with ESMTP id 4129C3DA059; Wed, 30 Jan 2008 17:24:40 +1300 (NZDT) Received: by kahikatea.snap.net.nz (Postfix, from userid 1000) id E1DD98FC6D; Wed, 30 Jan 2008 17:24:36 +1300 (NZDT) From: Nick Roberts MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-ID: <18335.64516.8815.911929@kahikatea.snap.net.nz> Date: Wed, 30 Jan 2008 04:25:00 -0000 To: Daniel Jacobowitz Cc: Andreas Schwab , Marc Khouzam , gdb@sources.redhat.com Subject: Re: -var-update using formatted value In-Reply-To: <20080130000415.GA25352@caradoc.them.org> References: <6D19CA8D71C89C43A057926FE0D4ADAA04290E1B@ecamlmw720.eamcs.ericsson.se> <18311.60638.724524.220449@kahikatea.snap.net.nz> <20080111225928.GA26360@caradoc.them.org> <18311.65093.38930.103045@kahikatea.snap.net.nz> <20080111235219.GA29698@caradoc.them.org> <18315.589.840507.551878@kahikatea.snap.net.nz> <20080129212632.GB17544@caradoc.them.org> <18335.47960.519547.477953@kahikatea.snap.net.nz> <20080130000415.GA25352@caradoc.them.org> X-Mailer: VM 7.19 under Emacs 23.0.60.5 X-IsSubscribed: yes Mailing-List: contact gdb-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sourceware.org X-SW-Source: 2008-01/txt/msg00327.txt.bz2 Daniel Jacobowitz writes: > On Wed, Jan 30, 2008 at 12:48:40PM +1300, Nick Roberts wrote: > > Both changes, i.e., printcmd.c and c-exp.y? > > Right. > > > Presumably if the change to c-exp.y is committed, there should be similar > > changes to ada-exp.y, f-exp.y etc. Does it not matter that this prefix > > is not part of the underlying language? > > Right. As long as it doesn't conflict with the underlying language, > no, it doesn't matter - GDB tries to accept the whole source language, > but also accepts other things. I wouldn't know if it did conflict, but unless I hear otherwise I'll assume it doesn't. In addition to c-exp.y, I'll amend: ada-exp.y f-exp.y jv-exp.y m2-exp.y objc-exp.y p-exp.y and write some tests. However, I can't run the testsuite for all these as I don't have the compilers on my system. The manual describes how integer constants should be typed for the various languages under the Languages node but not how they are printed with the various output formats under Examining. I guess they should be changed in both places. Since the printcmd.c patch only has to be done in one place, I'll start with that. > > Also, I think only the change to printcmd.c is necessary to solve Marcs bug > > report about -var-update. > > I think that report is covered by two of the other patches I reviewed > today. I don't think so. The other patches cover the wrong format not detecting a change in value, e.g. 1 -> 1.5 as decimal. This change covers a change in format masking a change in value. Marc's example was 11 -> 3 = 11 (binary). -- Nick http://www.inet.net.nz/~nickrob