On 2012-Oct-3 08:00 , Andreas Färber wrote: >> >+ if( ustr[1] < 0x20 || ustr[1] > 0x80 ) > ASCII didn't have the upper bit, so >= 0x80 would be more correct. > Does this correspond to any "upstream" that we should patch as well? Since rubout (0x7f) is also not representable, "ustr[1] >= 0x7f" might be even better.