Skip to content
This repository has been archived by the owner on Mar 1, 2023. It is now read-only.

Test “mp_read_double(mp_encode_double(-5.555)) check result” fails on x86 #17

Open
jirutka opened this issue Jul 17, 2017 · 1 comment

Comments

@jirutka
Copy link

jirutka commented Jul 17, 2017

msgpuck 2.0 with backported 40e24cc.

Test mp_read_double(mp_encode_double(-5.555)) check result fails on x86, but not x86_64.

    ok 61 - mp_read_double(mp_encode_uint(123)) check success
    ok 62 - mp_read_double(mp_encode_uint(123)) check pos advanced
    ok 63 - mp_read_double(mp_encode_uint(123)) check result
    ok 64 - mp_read_double(mp_encode_uint(12345)) check success
    ok 65 - mp_read_double(mp_encode_uint(12345)) check pos advanced
    ok 66 - mp_read_double(mp_encode_uint(12345)) check result
    ok 67 - mp_read_double(mp_encode_uint(123456789)) check success
    ok 68 - mp_read_double(mp_encode_uint(123456789)) check pos advanced
    ok 69 - mp_read_double(mp_encode_uint(123456789)) check result
    ok 70 - mp_read_double(mp_encode_uint(1234567890000ULL)) check success
    ok 71 - mp_read_double(mp_encode_uint(1234567890000ULL)) check pos advanced
    ok 72 - mp_read_double(mp_encode_uint(1234567890000ULL)) check result
    ok 73 - mp_read_double(mp_encode_uint(123456789123456789ULL)) check fail
    ok 74 - mp_read_double(mp_encode_uint(123456789123456789ULL)) check pos unchanged
    ok 75 - mp_read_double(mp_encode_int(-123)) check success
    ok 76 - mp_read_double(mp_encode_int(-123)) check pos advanced
    ok 77 - mp_read_double(mp_encode_int(-123)) check result
    ok 78 - mp_read_double(mp_encode_int(-12345)) check success
    ok 79 - mp_read_double(mp_encode_int(-12345)) check pos advanced
    ok 80 - mp_read_double(mp_encode_int(-12345))    #   Failed test 'mp_read_double(mp_encode_double(-5.555)) check result'
    #   in /home/buildozer/aports/testing/msgpuck/src/msgpuck-2.0/test/msgpuck.c at line 1051
 check result
    ok 81 - mp_read_double(mp_encode_int(-123456789)) check success
    ok 82 - mp_read_double(mp_encode_int(-123456789)) check pos advanced
    ok 83 - mp_read_double(mp_encode_int(-123456789)) check result
    ok 84 - mp_read_double(mp_encode_int(-1234567890000LL)) check success
    ok 85 - mp_read_double(mp_encode_int(-1234567890000LL)) check pos advanced
    ok 86 - mp_read_double(mp_encode_int(-1234567890000LL)) check result
    ok 87 - mp_read_double(mp_encode_int(-123456789123456789LL)) check fail
    ok 88 - mp_read_double(mp_encode_int(-123456789123456789LL)) check pos unchanged
    ok 89 - mp_read_double(mp_encode_float(6.565e6)) check success
    ok 90 - mp_read_double(mp_encode_float(6.565e6)) check pos advanced
    ok 91 - mp_read_double(mp_encode_float(6.565e6)) check result
    ok 92 - mp_read_double(mp_encode_double(-5.555)) check success
    ok 93 - mp_read_double(mp_encode_double(-5.555)) check pos advanced
-> not ok 94 - mp_read_double(mp_encode_double(-5.555)) check result <-
        # Looks like you failed 1 test of 96 run.
#   Failed test 'subtests'
#   in /home/buildozer/aports/testing/msgpuck/src/msgpuck-2.0/test/test.c at line 85
# Looks like you failed 1 test of 20 run.
ok 95 - mp_read_double(mp_encode_strl(100)) check fail
    ok 96 - mp_read_double(mp_encode_strl(100)) check pos unchanged
    # *** test_numbers: done ***
not ok 19 - subtests
algitbot pushed a commit to alpinelinux/aports that referenced this issue Jul 17, 2017
One test fails:

    ok 91 - mp_read_double(mp_encode_float(6.565e6)) check result
    ok 92 - mp_read_double(mp_encode_double(-5.555)) check success
    ok 93 - mp_read_double(mp_encode_double(-5.555)) check pos advanced
    not ok 94 - mp_read_double(mp_encode_double(-5.555)) check result
        # Looks like you failed 1 test of 96 run.
    #   Failed test 'subtests'
    #   in /home/buildozer/aports/testing/msgpuck/src/msgpuck-2.0/test/test.c at line 85
    # Looks like you failed 1 test of 20 run.
    ok 95 - mp_read_double(mp_encode_strl(100)) check fail
    ok 96 - mp_read_double(mp_encode_strl(100)) check pos unchanged
    # *** test_numbers: done ***
    not ok 19 - subtests

Reported to upstream: rtsisyk/msgpuck#17
@rtsisyk
Copy link
Owner

rtsisyk commented Jul 17, 2017

OK, I'll try to fix it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants