This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [Patch] objdump/xcoff: allow missing string table length

From: Tristan Gingold <gingold at adacore dot com>
To: Nick Clifton <nickc at redhat dot com>
Cc: binutils Development <binutils at sourceware dot org>
Date: Thu, 9 Jun 2011 17:29:30 +0200
Subject: Re: [Patch] objdump/xcoff: allow missing string table length
References: <9A1CBDED-09E1-4D83-9040-B910D96D7708@adacore.com> <4DF0D024.8090704@redhat.com>

On Jun 9, 2011, at 3:52 PM, Nick Clifton wrote:

> Hi Tristan,
> 
>> binutils/
>> 2011-06-09  Tristan Gingold<gingold@adacore.com>
>> 
>> 	* od-xcoff.c (xcoff32_read_symbols): Allow missing string table length.
> 
> Approved - please apply, although ...
> 
>> +      non_fatal (_("cannot read strings table len"));
> 
> Would you mind replacing "len" with "length", just to improve the quality of the warning message.

Sure, committed by the suggested change.

Tristan.

References:
- [Patch] objdump/xcoff: allow missing string table length
  - From: Tristan Gingold
- Re: [Patch] objdump/xcoff: allow missing string table length
  - From: Nick Clifton

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]