rtnl_cmp: Don't use rational type for intermediate result

See 4fbe2e349908a53032b3f23c2180a46e4f770c92.

1 files changed, 5 insertions, 8 deletions

diff --git a/libcrystfel/src/rational.c b/libcrystfel/src/rational.c
index 2cedf246..800decf5 100644
--- a/libcrystfel/src/rational.c
+++ b/libcrystfel/src/rational.c
@@ -169,16 +169,13 @@ Rational rtnl_sub(Rational a, Rational b)
 /* -1, 0 +1 respectively for a<b, a==b, a>b */
 signed int rtnl_cmp(Rational a, Rational b)
 {
-	Rational trt1, trt2;
-
-	trt1.num = a.num * b.den;
-	trt2.num = b.num * a.den;
+	long long int a_num, b_num;
 
-	trt1.den = a.den * b.den;
-	trt2.den = a.den * b.den;
+	a_num = a.num * b.den;
+	b_num = b.num * a.den;
 
-	if ( trt1.num > trt2.num ) return +1;
-	if ( trt1.num < trt2.num ) return -1;
+	if ( a_num > b_num ) return +1;
+	if ( a_num < b_num ) return -1;
 	return 0;
 }