Telodendria/Cytoplasm
Telodendria/Cytoplasm
3
3
Fork 5
Code Issues 17 Pull requests 2 Projects Releases 2 Packages Activity Actions

Ignore special case in Int64Str. This is hack but good enough for now.

Browse source
This commit is contained in:
Jordan Bancino 2023-08-13 12:33:18 +00:00
parent a1da7c7b4a
commit bcfab52f09
1 changed files with 34 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

38
src/Int64.c
View file

@ -26,6 +26,8 @@
#include <stddef.h> #include <stddef.h>
#include <signal.h> #include <signal.h>
#include <Log.h>
#ifdef INT64_NATIVE #ifdef INT64_NATIVE
#define Int64Sign(x) ((int) (((UInt64) (x)) >> 63)) #define Int64Sign(x) ((int) (((UInt64) (x)) >> 63))
#else #else
@ -44,17 +46,39 @@ Int64Str(Int64 x, int base, char *out, size_t len)
Int64 base64 = Int64Create(0, base); Int64 base64 = Int64Create(0, base);
if (neg)
{
x = Int64Neg(x);
}
/* We only have symbols up to base 16 */ /* We only have symbols up to base 16 */
if (base < 2 || base > 16) if (base < 2 || base > 16)
{ {
return 0; return 0;
} }
/*
* This algorithm doesn't work on INT64_MIN.
*
* But it works on all other integers in the range, so we
* just scoot the range in by one for now. It's a hack and
* I'm not a huge fan of it, but this function is mostly
* used in Json, which shouldn't have a range this large
* anyway (Json is limited to -2^53 -> 2^53-1).
*
* Proper fixes are always welcome.
*/
if (Int64Eq(x, Int64Create(0x80000000, 0x00000000)))
{
x = Int64Add(x, Int64Create(0, 1));
}
#if 0
else if (Int64Eq(x, Int64Create(0x7FFFFFFF, 0xFFFFFFFF)))
{
x = Int64Sub(x, Int64Create(0, 1));
}
#endif
if (base != 2 && base != 8 && base != 16 && neg)
{
x = Int64Neg(x);
}
do do
{ {
Int64 mod = Int64Rem(x, base64); Int64 mod = Int64Rem(x, base64);
@ -65,13 +89,13 @@ Int64Str(Int64 x, int base, char *out, size_t len)
x = Int64Div(x, base64); x = Int64Div(x, base64);
} while (Int64Gt(x, Int64Create(0, 0))); } while (Int64Gt(x, Int64Create(0, 0)));
if (base != 2 && base != 8 && base != 16)
{
/* /*
* Binary, octal, and hexadecimal are known to * Binary, octal, and hexadecimal are known to
* be bit representations. Everything else (notably * be bit representations. Everything else (notably
* decimal) should include the negative sign. * decimal) should include the negative sign.
*/ */
if (base != 2 && base != 8 && base != 16)
{
if (neg) if (neg)
{ {
out[i] = '-'; out[i] = '-';