On Sat, 15 Feb 2025 23:32:37 -0500
David Malcolm <dmalc...@redhat.com> wrote:
> > +static bool
> > +is_word_char( char ch ) {
> > + switch(ch) {
> > + case '0' ... '9':
> > + case 'a' ... 'z':
> > + case 'A' ... 'Z':
> > + case '$':
> > + case '-':
> > + case '_':
> > + return true;
> > + }
> > + return false;
> > +}
>
> Range based cases are a GCC extension IIRC, so this isn?t compatible
> with other C++ compilers
You recall correctly. I thought I was being clever. I will recast with
ISALNUM from libiberty.
> > + gcc_assert(false);
>
> This shows up in various places; use gcc_unreachable(); for this.
Did not know, thanks.
--jkl
