if(r < 0x80 && strchr("`^#*[]=|\\?${}()'<>&;", r))
                return 1;

Wouldn't this be clearer as

if(utfrune("`^#*[]=|\\?${}()'<>&;", r))
   return 1;

That's essentially what the r<0x80 (Runesync) does, isn't it?

Micah

Reply via email to