shithub: sl

Info  •  Files  •  Log  •  Branches

cmp_lt: return false if any is nan

--- a/meson.build

+++ b/meson.build

@@ -15,6 +15,7 @@

 	#'-Wconversion',

 	#'-Wfloat-equal',

 	#'-Wsign-conversion',

+	'-Wpedantic',

 	'-Waggregate-return',

 	'-Werror=odr',

 	'-Werror=strict-aliasing',

--- a/src/operators.c

+++ b/src/operators.c

@@ -155,9 +155,12 @@

 	double da = conv_to_double(a, atag);

 	double db = conv_to_double(b, btag);

+	if(isnan(da) || isnan(db))

+		return false;

+

 	// casting to double will only get the wrong answer for big int64s

 	// that differ in low bits

-	if(da < db && !isnan(da) && !isnan(db))

+	if(da < db)

 		return true;

 	if(db < da)

 		return false;