Re: [Groff] Patch for two grohtml bug fixes..

[Gaius Mulley]

> Hi Gaius,
> 
> > Also modified is the constant 65535 to MAX_COLOR_VAL, for what it is
> > worth.. :-)
> > 
> > ...
> > 
> > -  if (val > 65536) {
> > +  if (val > color::MAX_COLOR_VAL) {
> >      warning(WARN_RANGE, "%1 cannot be greater than 1", color);
> > -    return 65535;      // this is 0xffff
> > +    return color::MAX_COLOR_VAL;
> 
> The two constants replaced by color::MAX_COLOR_VAL are different.  This
> looks OK because the old code looked `out by one' but I thought it
> worth raising to make sure as I haven't looked at the code at all, just
> the diff.

Hi Ralph,

aaahhgg, yes a bug sorry Werner. Well spotted Ralph, this should
either be color::MAX_COLOR_VAL+1 or just use the literal constants as
Werner had previously. Perhaps with hindsight the literals make more
sense anyway in this context as 'f' is not always associated with
'color',

Gaius