From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1165 invoked by alias); 22 Apr 2003 16:55:05 -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 1154 invoked from network); 22 Apr 2003 16:55:04 -0000 Received: from unknown (HELO crack.them.org) (65.125.64.184) by sources.redhat.com with SMTP; 22 Apr 2003 16:55:04 -0000 Received: from nevyn.them.org ([66.93.61.169] ident=mail) by crack.them.org with asmtp (Exim 3.12 #1 (Debian)) id 198130-0004in-00; Tue, 22 Apr 2003 11:55:11 -0500 Received: from drow by nevyn.them.org with local (Exim 3.36 #1 (Debian)) id 19812l-0008Fm-00; Tue, 22 Apr 2003 12:54:55 -0400 Date: Tue, 22 Apr 2003 16:55:00 -0000 From: Daniel Jacobowitz To: Andrew Cagney Cc: Jason Molenda , gdb-patches@sources.redhat.com Subject: Re: PATCH: Add type_sprint() function to return type in string form Message-ID: <20030422165455.GA31686@nevyn.them.org> Mail-Followup-To: Andrew Cagney , Jason Molenda , gdb-patches@sources.redhat.com References: <20030418152426.A93348@molenda.com> <20030422032629.GB5033@nevyn.them.org> <3EA57249.5020104@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <3EA57249.5020104@redhat.com> User-Agent: Mutt/1.5.1i X-SW-Source: 2003-04/txt/msg00400.txt.bz2 On Tue, Apr 22, 2003 at 12:48:09PM -0400, Andrew Cagney wrote: > >On Fri, Apr 18, 2003 at 03:24:26PM -0700, Jason Molenda wrote: > > > >>There are a few places in gdb where code prints an error message > >>with a type included in it. The only way for these functions to > >>print a type is with type_print(), which takes a ui_file stream to > >>print its output to. This means they either have to send the output > >>to gdb_stderr, or build up a fake memory ui_file and retrieve the > >>contents. > >> > >>This patch adds a type_sprint() which does the latter and returns the > >>xmalloc()'ed string. > > Good interface choice! Just some tweaks. > > sprint makes me think of the nasty sprintf family, which this is > definitly not. So .. suggest calling it something like type_xstrdup: > x-> xmalloc family; strdup -> allocate a string duplicate (like > ui_file_xstrdup, but yes pushing it a bit). The alternative would be > type_xasprint (&string, ...) but I think that is getting ugly :-) How about type_asprint? I really don't think that the x is necessary, but the a would be a convenient reminder. > The old error message should never have included a period ("."), might > as well remove it while tweaking the code. Didn't we settle on full sentence error messages the last time this came up? -- Daniel Jacobowitz MontaVista Software Debian GNU/Linux Developer