diff options
| author | jan.nijtmans <nijtmans@users.sourceforge.net> | 2016-11-17 16:26:50 (GMT) |
|---|---|---|
| committer | jan.nijtmans <nijtmans@users.sourceforge.net> | 2016-11-17 16:26:50 (GMT) |
| commit | d31be24ebfb55014fd29140d4406c8a840845763 (patch) | |
| tree | 8745c9b69988bfd839bd9d090d41713bd840c002 /generic/tclObj.c | |
| parent | 3dd86e6ebc0137c3a2c02d3a046de046571e3789 (diff) | |
| parent | 205419d4b7c3801fe37ae41bceba18fc3b230308 (diff) | |
| download | tcl-d31be24ebfb55014fd29140d4406c8a840845763.zip tcl-d31be24ebfb55014fd29140d4406c8a840845763.tar.gz tcl-d31be24ebfb55014fd29140d4406c8a840845763.tar.bz2 | |
Fix libtommath's mp_radix_size() function such that it returns 2 for single-digit numbers. Add testcases for mp_radix_size() and mp_iseven(). Undo useless change in bn_mp_add_d.c (bring back libtommath's version).
Diffstat (limited to 'generic/tclObj.c')
| -rw-r--r-- | generic/tclObj.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/generic/tclObj.c b/generic/tclObj.c index d3f59ec..6c850af 100644 --- a/generic/tclObj.c +++ b/generic/tclObj.c @@ -3252,13 +3252,11 @@ UpdateStringOfBignum( if (status != MP_OKAY) { Tcl_Panic("radix size failure in UpdateStringOfBignum"); } - if (size == 3) { + if (size < 2) { /* - * mp_radix_size() returns 3 when more than INT_MAX bytes would be + * mp_radix_size() returns < 2 when more than INT_MAX bytes would be * needed to hold the string rep (because mp_radix_size ignores - * integer overflow issues). When we know the string rep will be more - * than 3, we can conclude the string rep would overflow our string - * length limits. + * integer overflow issues). * * Note that so long as we enforce our bignums to the size that fits * in a packed bignum, this branch will never be taken. |