From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-56946-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 15819 invoked by alias); 28 Jun 2008 18:17:21 -0000
Received: (qmail 15805 invoked by uid 22791); 28 Jun 2008 18:17:20 -0000
X-Spam-Check-By: sourceware.org
Received: from mtaout5.012.net.il (HELO mtaout5.012.net.il) (84.95.2.13)     by sourceware.org (qpsmtpd/0.31) with ESMTP; Sat, 28 Jun 2008 18:16:33 +0000
Received: from HOME-C4E4A596F7 ([80.230.31.196]) by i_mtaout5.012.net.il (HyperSendmail v2004.12) with ESMTPA id <0K3600K8URGCMED1@i_mtaout5.012.net.il> for gdb-patches@sources.redhat.com; Sat, 28 Jun 2008 21:31:24 +0300 (IDT)
Date: Sat, 28 Jun 2008 18:23:00 -0000
From: Eli Zaretskii <eliz@gnu.org>
Subject: Re: [MI non-stop 04/11] Implement --thread and --frame.
In-reply-to: <g45snv$569$2@ger.gmane.org>
X-012-Sender: halo1@inter.net.il
To: Vladimir Prus <vladimir@codesourcery.com>
Cc: gdb-patches@sources.redhat.com
Reply-to: Eli Zaretskii <eliz@gnu.org>
Message-id: <u8wwpctnd.fsf@gnu.org>
References: <200806282044.14246.vladimir@codesourcery.com> <uej6hcw5i.fsf@gnu.org> <g45snv$569$2@ger.gmane.org>
X-IsSubscribed: yes
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
X-SW-Source: 2008-06/txt/msg00560.txt.bz2

> From:  Vladimir Prus <vladimir@codesourcery.com>
> Date:  Sat, 28 Jun 2008 21:34:57 +0400
> 
> Eli Zaretskii wrote:
> 
> >> From: Vladimir Prus <vladimir@codesourcery.com>
> >> Date: Sat, 28 Jun 2008 20:44:14 +0400
> >> 
> >> 
> >> +  if (parse->frame != -1 && !parse->thread == -1)
> >> +    error ("Cannot specify --frame without --thread");
> > 
> > Why is this error message not in _(), while all the rest are?
> > 
> > Btw, do we at all want error messages issued by MI commands
> > translated?  
> 
> I think we don't, but I'm not sure.
> 
> 
> > MI commands are invoked by a program, so error messages 
> > we generate should be understandable by a program, which probably
> > means they should not be translated.
> 
> It's a bit questionable. For example, the error you mention above
> is clearly a bug in frontend. Presenting a translated version of
> that message to the user is essentially pointless. On the other
> hand, "Thread is running", or "Memory not accessible" messages
> can be helpful for users. Do we need two error messages, maybe?

Maybe.  Do front ends show the error messages to the user, or do they
act on them themselves (or both)?

> >> +      if (strncmp (chp, "--thread", 8) == 0)
> > 
> > Please, let's not use literal constants in this context, let's use
> > sizeof instead.
> 
> sizeof? For all I know, sizeof("--thread") will be wrong here.

Why would it be wrong?

> >> +      parse->frame = strtol (chp, &chp, 10);
> > 
> > Do we really want to disallow non-decimal numbers here?  What about
> > hex frame numbers?
> 
> Why would frontend want to specify frame level in hex?

I dunno, does the MI spec mandate decimal here?  If it does, then my
comment is hereby withdrawn.