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