Re: [PATCH] genmodes: don't truncate a mode name of length = 7

2012-04-20 Thread Richard Guenther
On Thu, Apr 19, 2012 at 10:35 PM, Jim Meyering j...@meyering.net wrote: Richard Guenther wrote: Sure, my point was that the       if (strlen (m-name) = sizeof buf)         {           error (%s:%d:mode name \%s\ is too long,                  m-file, m-line, m-name);           continue;    

Re: [PATCH] genmodes: don't truncate a mode name of length = 7

2012-04-20 Thread Jim Meyering
Richard Guenther wrote: On Thu, Apr 19, 2012 at 10:35 PM, Jim Meyering j...@meyering.net wrote: Richard Guenther wrote: Sure, my point was that the       if (strlen (m-name) = sizeof buf)         {           error (%s:%d:mode name \%s\ is too long,                  m-file, m-line, m-name);

[PATCH] genmodes: don't truncate a mode name of length = 7

2012-04-19 Thread Jim Meyering
Richard Guenther wrote: Sure, my point was that the if (strlen (m-name) = sizeof buf) { error (%s:%d:mode name \%s\ is too long, m-file, m-line, m-name); continue; } check does not account for the (conditional) prepending of 'C'