From 01cdd76377faf07132495692feab50a1421ab94a Mon Sep 17 00:00:00 2001
From: bakaq <bakaq@bakaq.net>
Date: Tue, 20 Aug 2024 14:37:10 -0300
Subject: [PATCH] Fix rational numbers serialization

---
 src/machine/lib_machine.rs    |  2 +-
 src/machine/parsed_results.rs | 22 ++++++++++++++--------
 2 files changed, 15 insertions(+), 9 deletions(-)

diff --git a/src/machine/lib_machine.rs b/src/machine/lib_machine.rs
index ab41ff01d..10bec91f1 100644
--- a/src/machine/lib_machine.rs
+++ b/src/machine/lib_machine.rs
@@ -647,7 +647,7 @@ mod tests {
         let result = machine.run_query(query);
 
         let expected = Value::Structure(
-            // Composite term
+            // Compound term
             "a".into(),
             vec![
                 Value::String("asdf".into()), // String
diff --git a/src/machine/parsed_results.rs b/src/machine/parsed_results.rs
index 81bad1f35..d48a37f82 100644
--- a/src/machine/parsed_results.rs
+++ b/src/machine/parsed_results.rs
@@ -166,8 +166,8 @@ impl From<&QueryResolution> for Value {
 }
 
 struct ValueRationalInner {
-    up: IBig,
-    down: UBig,
+    numerator: IBig,
+    denominator: UBig,
 }
 
 impl Serialize for ValueRationalInner {
@@ -176,13 +176,13 @@ impl Serialize for ValueRationalInner {
         S: Serializer,
     {
         let mut seq = serializer.serialize_seq(Some(2))?;
-        match self.up.to_i64() {
+        match self.numerator.to_i64() {
             Some(small) => seq.serialize_element(&small)?,
-            None => seq.serialize_element(&self.up.to_string())?,
+            None => seq.serialize_element(&self.numerator.to_string())?,
         }
-        match self.down.to_i64() {
+        match self.denominator.to_u64() {
             Some(small) => seq.serialize_element(&small)?,
-            None => seq.serialize_element(&self.down.to_string())?,
+            None => seq.serialize_element(&self.denominator.to_string())?,
         }
         seq.end()
     }
@@ -203,9 +203,15 @@ impl Serialize for Value {
                 }
             },
             Value::Rational(r) => {
-                let (up, down) = r.clone().into_parts();
+                let (numerator, denominator) = r.clone().into_parts();
                 let mut map = serializer.serialize_map(Some(1))?;
-                map.serialize_entry("rational", &ValueRationalInner { up, down })?;
+                map.serialize_entry(
+                    "rational",
+                    &ValueRationalInner {
+                        numerator,
+                        denominator,
+                    },
+                )?;
                 map.end()
             }
             Value::Float(f) => serializer.serialize_f64(f.into_inner()),