Heikki Linnakangas <hlinnakangas@vmware.com> writes:
> A regex with that many different colors is an extreme case, so I think
> it's enough to turn the assertion in newcolor() into a run-time check,
> and throw a "too many colors in regexp" error. Alternatively, we could
> expand 'color' from short to int, but that would double the memory usage
> of sane regexps with less different characters.
Obviously Henry didn't think that far ahead. I agree that throwing
an error is the best solution, and that widening "color" is probably
not what we want to do. You want to fix that, or shall I?
regards, tom lane
