From b3002bbed47dcc7430bd9ca5623ac9e064a9b006 Mon Sep 17 00:00:00 2001
From: Armin Ronacher <armin.ronacher@active-4.com>
Date: Mon, 16 Sep 2024 19:35:55 +0200
Subject: [PATCH] Add an Value::is_integer method (#580)

---
 CHANGELOG.md               |  2 ++
 minijinja/src/tests.rs     |  8 +-------
 minijinja/src/value/mod.rs | 11 +++++++++++
 3 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 32e5d07e..fa71603c 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -14,6 +14,8 @@ All notable changes to MiniJinja are documented here.
   values.  #571
 - Changed sort order of `Ord` to avoid accidentally non total order
   that could cause panics on Rust 1.81.  #579
+- Added a `Value::is_integer` method to allow a user to tell floats
+  and true integers apart.  #580
 
 ## 2.2.0
 
diff --git a/minijinja/src/tests.rs b/minijinja/src/tests.rs
index 0a648535..f3528a82 100644
--- a/minijinja/src/tests.rs
+++ b/minijinja/src/tests.rs
@@ -307,13 +307,7 @@ mod builtins {
     /// ```
     #[cfg_attr(docsrs, doc(cfg(feature = "builtins")))]
     pub fn is_integer(v: Value) -> bool {
-        matches!(
-            v.0,
-            crate::value::ValueRepr::U64(_)
-                | crate::value::ValueRepr::I64(_)
-                | crate::value::ValueRepr::U128(_)
-                | crate::value::ValueRepr::I128(_)
-        )
+        v.is_integer()
     }
 
     /// Checks if this value is a float
diff --git a/minijinja/src/value/mod.rs b/minijinja/src/value/mod.rs
index 4fdac445..8fc43acf 100644
--- a/minijinja/src/value/mod.rs
+++ b/minijinja/src/value/mod.rs
@@ -979,6 +979,17 @@ impl Value {
         )
     }
 
+    /// Returns true if the number is a real integer.
+    ///
+    /// This can be used to distinguish `42` from `42.0`.  For the most part
+    /// the engine keeps these the same.
+    pub fn is_integer(&self) -> bool {
+        matches!(
+            self.0,
+            ValueRepr::U64(_) | ValueRepr::I64(_) | ValueRepr::I128(_) | ValueRepr::U128(_)
+        )
+    }
+
     /// Returns `true` if the map represents keyword arguments.
     pub fn is_kwargs(&self) -> bool {
         Kwargs::extract(self).is_some()