Re: bignum branch

[Andy Moreton]

On Wed 08 Aug 2018, Tom Tromey wrote:

>>>>>> "Andy" == Andy Moreton <address@hidden> writes:
>
> Andy> Have you run "make check" on the bignum branch ?
>
> I did this and fixed the problems I saw.
>
> Andy> 1) A crash in the json tests
> Andy>    This is bug#32381, fixed on master in commit 3eac378c96.
>
> I didn't see this, but since it is unrelated to the branch, I wouldn't
> have anyway.

Tis only happens if you configure with --with-json to use the jansson
library for JSON parsing.

> Andy> 2) An eassert in make_bignum_str called from string_to_number.
> Andy>    This appears to be caused by differences in the syntax accepted by
> Andy>    emacs lisp and by mpz_set_str.  To reproduce:
> ELISP> (format "+%S" (- most-negative-fixnum))
> Andy>      "+2305843009213693952"
> ELISP> (string-to-number "+2305843009213693952")
> Andy>      ### crash ###
>
> I fixed this.

Thanks - retesting on Windows mingw64 x86_64 shows this is fixed.

> Andy> 3) Another crash which I have not yet had time to analyse with gdb.

This crashes in data-tests-logcount from test/src/data-tests.el, so there is
a problem in logcount.

> I didn't see this.  However, emacs-module-tests.el had a test failure;
> I've updated the test.
>
> I've pushed all the patches I have.

Once the logcount problem is fixed, this is looking ready to merge.

    AndyM