search

picrin-scheme / picrin

lightweight scheme interpreter
MIT License
414 stars 35 forks source link

Restrict char in the range of ASCII. #337

Closed omasanori closed 8 years ago

omasanori commented 8 years ago

picrin accepts the code (char->integer (integer->char 128)) but the result is machine-dependent, since the C char type can be either signed or unsigned as permitted by the spec. Unfortunately, the result of signed one is incompatible with Unicode-aware implementations. For interoperability, we should deny the range beyond ASCII for now.

Ref: #211

nyuichi commented 8 years ago

Sorry for the late response. It looks good to me.

omasanori commented 8 years ago

Thanks!