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] Fix grammar in bfd warning message


On Tue, 2016-07-12 at 12:02 +0200, Douglas B Rupp wrote:
> Please review and apply attached if you please.
> I don't have write privileges.

> -             (_("Warning: Writing section `%s' to huge (ie negative)
file offset 0x%lx."),
> +             (_("Warning: Writing section `%s' too huge (ie
negative) file offset 0x%lx."),

I don't think this change is an improvement.  In the original phrasing
"to" was intended to be a preposition: it's complaining that it is
writing a section "to" a location in the file whose offset is so large
that it doesn't fit in a signed int.  With your change the sentence no
longer has a preposition which breaks the grammar. 

If you wanted to remove the potential ambiguity in the original
sentence then replacing "to" with "at" might do it.  

On the other hand, warning about offsets that don't fit in 31 bits
seems a tiny bit arbitrary and I don't know if that diagnostic is
actually conveying any useful information in the first place.

p.


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