Re: [PATCH v3] gdb/hppa: guess g packet size

[Sven Schnelle <svens@stackframe.org>]

Simon Marchi <simark@simark.ca> writes:

>> +	  warning (_("target returned a target description this gdb doesn't "
>> +		     "support for the HP/PA architecture"));
>
> Good idea to put out a warning.  I would suggesting some minor changes
> to the message:
>
> 	  warning (_("The target returned a target description but this gdb "
> 		     "doesn't support target descriptions for the HP/PA
> 		     "architecture"));
>
> Just thinking out loud: it would actually be nice to have this warning
> for all architectures that don't support target descriptions, if there
> was a way to put it at a single common place.
>
> Other than that, the patch looks fine.
>
> Approved-By: Simon Marchi <simon.marchi@efficios.com>
>
> Do you have push access or would you like me to push the patch on your
> behalf (with the modification proposed above)?

I don't have push acess - would appreciate if you could adjust the
message and push it.