Omit a few minor unnecessary range checks

Based on Pip Cet’s review (Bug#36370#19).
* src/fileio.c (Fdo_auto_save):
* src/image.c (lookup_image):
* src/textprop.c (Fnext_single_char_property_change):
Prefer XFIXNUM to XFIXNAT for clarity and consistency with
neighboring code, and to avoid unnecessary range checks.
* src/image.c (lookup_image): Omit unnecessary range checks.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/fileio.c b/src/fileio.c
index 61e10dac47f..ed1d2aedf37 100644
--- a/src/fileio.c
+++ b/src/fileio.c
@@ -5852,7 +5852,7 @@ A non-nil CURRENT-ONLY argument means save only current buffer.  */)
                    spare the user annoying messages.  */
                 && XFIXNUM (BVAR (b, save_length)) > 5000
                 && (growth_factor * (BUF_Z (b) - BUF_BEG (b))
-                    < (growth_factor - 1) * XFIXNAT (BVAR (b, save_length)))
+                    < (growth_factor - 1) * XFIXNUM (BVAR (b, save_length)))
                 /* These messages are frequent and annoying for `*mail*'.  */
                 && !NILP (BVAR (b, filename))
                 && NILP (no_message))