Merge pull request #6546 from guybe7/fix_neg_zero

Make sure Redis does not reply with negative zero
This commit is contained in:
Salvatore Sanfilippo 2020-04-02 16:26:57 +02:00 committed by GitHub
commit 10b626b3d5
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 10 additions and 0 deletions

View File

@ -602,6 +602,10 @@ int ld2string(char *buf, size_t len, long double value, ld2string_mode mode) {
} }
if (*p == '.') l--; if (*p == '.') l--;
} }
if (l == 2 && buf[0] == '-' && buf[1] == '0') {
buf[0] = '0';
l = 1;
}
break; break;
default: return 0; /* Invalid mode. */ default: return 0; /* Invalid mode. */
} }

View File

@ -151,4 +151,10 @@ start_server {tags {"incr"}} {
catch {r incrbyfloat foo 1} err catch {r incrbyfloat foo 1} err
format $err format $err
} {ERR*valid*} } {ERR*valid*}
test {No negative zero} {
r incrbyfloat foo [expr double(1)/41]
r incrbyfloat foo [expr double(-1)/41]
r get foo
} {0}
} }