[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)

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 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->lin

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 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); >>           continu