On Mon, May 9, 2011 at 2:08 PM, Doug Evans wrote: > On Mon, May 9, 2011 at 1:36 PM, Eli Zaretskii wrote: >>> Date: Mon, 9 May 2011 13:19:56 -0700 >>> From: Doug Evans >>> Cc: gdb-patches@sourceware.org >>> >>> > Why not just say "if your @command{iconv} program is installed in a >>> > non-standard place, you will need to tell @value{GDBN} where to find >>> > it."? >>> > >>> >>> The iconv program comes from glibc, but you're right, it doesn't >>> *necessarily* come from there. >>> >>> How about this? >> >> It's fine, but... >> >>> +If @value{GDBN} is using the @code{iconv} program and it is installed in >>> +a non-standard place, you will need to tell @value{GDBN} where to find it. >> >> The first "it" is ambiguous: it could have meant GDB or iconv. > > Yeah, I recognized the ambiguity in earlier versions and explicitly > tried a couple of variations to avoid the ambiguity. > I like what I came up with but it can always be improved of course. > >> Suggest a slight rephrase: >> >>  If @value{GDBN} is using the @code{iconv} program which is installed >>  in a non-standard place, ... > > Ok. > Here is the patch I checked in.