diff options
author | dkf <donal.k.fellows@manchester.ac.uk> | 2012-12-23 08:17:17 (GMT) |
---|---|---|
committer | dkf <donal.k.fellows@manchester.ac.uk> | 2012-12-23 08:17:17 (GMT) |
commit | 42c352d6258bc3ec26c19183c29b5a4ac4301a81 (patch) | |
tree | 904a5c784012083beffa88301ce90c0d9ca69bc6 /tests/format.test | |
parent | a40590fd31db326000458a35d4bff19a8f7a3b4d (diff) | |
download | tcl-42c352d6258bc3ec26c19183c29b5a4ac4301a81.zip tcl-42c352d6258bc3ec26c19183c29b5a4ac4301a81.tar.gz tcl-42c352d6258bc3ec26c19183c29b5a4ac4301a81.tar.bz2 |
Change back to using an isDigit function. We simply don't need to make any
(formally non-portable) assumptions about what happens when an unsigned zero
is decremented, and the code isn't in a performance-critical area.
Remark by jan.nijtmans: The macro is perfectly portable! Not portable is the
exact result of the substraction ('\xB0' - '0' might give 0x80 on some platforms and 0xffffff80 on others), but comparing <= 9 always gives the correct result. We are only checking for digits here! The macro correctly inlines with any compiler, so it's better anyway.
Remark by dkf: But it's less clear. In this code, that's more important than a teeny bit of speed from inlining in a non-critical location.
Diffstat (limited to 'tests/format.test')
0 files changed, 0 insertions, 0 deletions