At 2018-05-05T15:43:35+0200, Werner LEMBERG wrote:
> 
> >> > +  static int n_max = (HYPHEN_NOT_LAST_LINE | HYPHEN_NOT_LAST_CHARS
> >> > +    | HYPHEN_NOT_FIRST_CHARS | HYPHEN_LAST_CHAR
> >> > +    | HYPHEN_FIRST_CHAR);
> >> 
> >> s/static int/int const/?
> 
> Please use `const int' – there is no single instance of `int const' in
> the groff code.

Okay.  The Global Groff Const Relocation can wait for some point in the
indefinite future.  ;-)

> I prefer that.  It's always a bit problematic ORing enumeration values
> (I know how to circumvent that limitation, but...)

That's fair.  I'll cook up a revised patch for review.

-- 
Regards,
Branden

Attachment: signature.asc
Description: PGP signature

Reply via email to