From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21609 invoked by alias); 7 Jul 2007 17:06:20 -0000 Received: (qmail 21600 invoked by uid 22791); 7 Jul 2007 17:06:19 -0000 X-Spam-Check-By: sourceware.org Received: from NaN.false.org (HELO nan.false.org) (208.75.86.248) by sourceware.org (qpsmtpd/0.31) with ESMTP; Sat, 07 Jul 2007 17:06:16 +0000 Received: from nan.false.org (localhost [127.0.0.1]) by nan.false.org (Postfix) with ESMTP id 6597A982D6; Sat, 7 Jul 2007 17:06:15 +0000 (GMT) Received: from caradoc.them.org (22.svnf5.xdsl.nauticom.net [209.195.183.55]) by nan.false.org (Postfix) with ESMTP id 4429B982BB; Sat, 7 Jul 2007 17:06:15 +0000 (GMT) Received: from drow by caradoc.them.org with local (Exim 4.67) (envelope-from ) id 1I7DjZ-0005y0-5d; Sat, 07 Jul 2007 13:06:13 -0400 Date: Sat, 07 Jul 2007 17:06:00 -0000 From: Daniel Jacobowitz To: Eli Zaretskii Cc: gdb-patches@sourceware.org Subject: Re: [RFC] Changes to signed char and unsigned char handling Message-ID: <20070707170613.GA22650@caradoc.them.org> Mail-Followup-To: Eli Zaretskii , gdb-patches@sourceware.org References: <20070705135402.GA4300@caradoc.them.org> <20070706213408.GA5154@caradoc.them.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.15 (2007-04-09) 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: 2007-07/txt/msg00127.txt.bz2 On Sat, Jul 07, 2007 at 01:03:51PM +0300, Eli Zaretskii wrote: > > Date: Fri, 6 Jul 2007 17:34:08 -0400 > > From: Daniel Jacobowitz > > Cc: gdb-patches@sourceware.org > > > > > Didn't RMS object to this change? > > > > His objection was to the original change, which is currently in HEAD, > > to not print "unsigned char *" variables as strings. It was not an > > especially definitive objection; I think that this version is better > > than the behavior of earlier versions of GDB, which would satisfy his > > stated concern. > > Well, maybe we should ask for his opinion on your suggested patch. If it's all right with you, I'll leave this up to you. I've no complaints about asking his opinion, but I don't really think it's necessary either. > > We don't use `i' or `s'; the use of `i' or `s' in the user's format > > specification causes `display' to behave like `examine' instead of > > like `print'. How is this? > > > > in fact, @code{display} decides > > whether to use @code{print} or @code{x} depending on your format > > specification---@code{display} uses @code{x} if you specify a unit size, > > the @samp{i} format, or the @samp{s} format; otherwise it uses @code{print}. > > Aha, I see that I was doubly confused. So maybe this is more clear > yet: > > in fact, @code{display} decides > whether to use @code{print} or @code{x} depending on your format > specification---it uses @code{x} if you specify either the @samp{i} > or @samp{s} format, or a unit size; otherwise it uses @code{print}. This version's fine with me. Thanks again. -- Daniel Jacobowitz CodeSourcery