simon-git: putty (master): Simon Tatham
Commits to Tartarus hosted VCS
tartarus-commits at lists.tartarus.org
Mon Jan 7 20:08:40 GMT 2019
TL;DR:
9a596665 Add .rcpp files to .gitignore.
48f4b0a3 Correct the comment in mpint_i.h.
55ea49de Allow command-line override of the BignumInt type.
7fd81501 mpint: fix further integer-type confusions.
Repository: https://git.tartarus.org/simon/putty.git
On the web: https://git.tartarus.org/?p=simon/putty.git
Branch updated: master
Committer: Simon Tatham <anakin at pobox.com>
Date: 2019-01-07 20:08:40
commit 9a59666577d9d0bb2ab5d4b135f4876e879f27da
web diff https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=9a59666577d9d0bb2ab5d4b135f4876e879f27da;hp=f5576b26c3a593a72f333f42e69f38129d7c13b2
Author: Simon Tatham <anakin at pobox.com>
Date: Mon Jan 7 19:51:59 2019 +0000
Add .rcpp files to .gitignore.
This should have been done months ago in commit bf0cf984c, but I've
been indecisive about whether to keep my local dev builds in the
windows subdirectory itself or one level further down...
.gitignore | 1 +
1 file changed, 1 insertion(+)
commit 48f4b0a36cba5f9cad86137729a2345ae838ca56
web diff https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=48f4b0a36cba5f9cad86137729a2345ae838ca56;hp=9a59666577d9d0bb2ab5d4b135f4876e879f27da
Author: Simon Tatham <anakin at pobox.com>
Date: Mon Jan 7 07:20:54 2019 +0000
Correct the comment in mpint_i.h.
At some point in mpint.c development I switched the main macro defined
by the ifdefs from BIGNUM_INT_BITS to the new BIGNUM_INT_BITS_BITS, so
I could loop from 0 to the latter in safe bit-shift loops that test
each bit of a shift count. But I forgot to change the comment
accordingly.
mpint_i.h | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
commit 55ea49de1eebb5d05b3c24033d97dc140ac41e54
web diff https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=55ea49de1eebb5d05b3c24033d97dc140ac41e54;hp=48f4b0a36cba5f9cad86137729a2345ae838ca56
Author: Simon Tatham <anakin at pobox.com>
Date: Mon Jan 7 07:25:45 2019 +0000
Allow command-line override of the BignumInt type.
This makes it easy to re-test the mpint functions using different word
sizes and smoke out any more confusions between integer types in
mpint.c, by recompiling with -DBIGNUM_OVERRIDE=4 or =5 or =6 (for 16-,
32- or 64-bit respectively).
BIGNUM_OVERRIDE only lets you force the size downwards, not upwards:
of course the default behaviour is to use the largest BignumInt the
ifdefs can find a way to, and they can't magic up a bigger one just
because you tell them you'd like one.
mpint_i.h | 25 +++++++++++++++++++------
1 file changed, 19 insertions(+), 6 deletions(-)
commit 7fd815014e30fe0b17c264f2ea11247e190f300b
web diff https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=7fd815014e30fe0b17c264f2ea11247e190f300b;hp=55ea49de1eebb5d05b3c24033d97dc140ac41e54
Author: Simon Tatham <anakin at pobox.com>
Date: Mon Jan 7 20:04:06 2019 +0000
mpint: fix further integer-type confusions.
This makes the test suite pass if I compile with -DBIGNUM_OVERRIDE=4
to fall back to 16-bit BignumInt. In that mode, BignumInt is smaller
than 'int', which means default promotion keeps causing things to get
promoted to 'int' unexpectedly, so I had to add some casts back down.
mpint.c | 16 +++++++++++-----
1 file changed, 11 insertions(+), 5 deletions(-)
More information about the tartarus-commits
mailing list