From efb85ed5707ffb0a1ca0957b2349971f28fb2270 Mon Sep 17 00:00:00 2001
From: Tim Stack <timothyshanestack@gmail.com>
Date: Thu, 9 May 2024 15:41:30 -0700
Subject: [PATCH] [tests] try to address some test issues

---
 src/base/humanize.time.cc              |  9 +++++----
 src/lnav_commands.cc                   | 10 ++--------
 src/third-party/rapidyaml/ryml_all.hpp |  5 +++++
 test/test_cmds.sh                      |  3 +++
 4 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/src/base/humanize.time.cc b/src/base/humanize.time.cc
index a791a3db..5c7fd2aa 100644
--- a/src/base/humanize.time.cc
+++ b/src/base/humanize.time.cc
@@ -57,8 +57,9 @@ point::as_time_ago() const
         current_time.tv_sec = convert_log_time_to_local(current_time.tv_sec);
     }
 
-    auto delta
-        = std::chrono::seconds(current_time.tv_sec - this->p_past_point.tv_sec);
+    auto curr_secs = std::chrono::seconds(current_time.tv_sec);
+    auto past_secs = std::chrono::seconds(this->p_past_point.tv_sec);
+    auto delta = curr_secs - past_secs;
     if (delta < 0s) {
         return "in the future";
     }
@@ -113,8 +114,8 @@ point::as_precise_time_ago() const
             return fmt::format(FMT_STRING("{:2} seconds ago"), diff.tv_sec);
         }
 
-        time_t seconds = diff.tv_sec % 60;
-        time_t minutes = diff.tv_sec / 60;
+        lnav::time64_t seconds = diff.tv_sec % 60;
+        lnav::time64_t minutes = diff.tv_sec / 60;
 
         return fmt::format(FMT_STRING("{:2} minute{} and {:2} second{} ago"),
                            minutes,
diff --git a/src/lnav_commands.cc b/src/lnav_commands.cc
index 2b8351a6..56889403 100644
--- a/src/lnav_commands.cc
+++ b/src/lnav_commands.cc
@@ -316,15 +316,12 @@ com_unix_time(exec_context& ec,
         }
         if (parsed) {
             char ftime[128];
-            int len;
 
             strftime(ftime,
                      sizeof(ftime),
                      "%a %b %d %H:%M:%S %Y  %z %Z",
                      localtime_r(&u_time, &log_time));
-            len = strlen(ftime);
-            snprintf(ftime + len, sizeof(ftime) - len, " -- %ld", u_time);
-            retval = std::string(ftime);
+            retval = fmt::format(FMT_STRING("{} -- {}"), ftime, u_time);
         } else {
             return ec.make_error("invalid unix time -- {}", args[1]);
         }
@@ -662,7 +659,6 @@ com_current_time(exec_context& ec,
     struct tm localtm;
     std::string retval;
     time_t u_time;
-    size_t len;
 
     memset(&localtm, 0, sizeof(localtm));
     u_time = time(nullptr);
@@ -670,9 +666,7 @@ com_current_time(exec_context& ec,
              sizeof(ftime),
              "%a %b %d %H:%M:%S %Y  %z %Z",
              localtime_r(&u_time, &localtm));
-    len = strlen(ftime);
-    snprintf(ftime + len, sizeof(ftime) - len, " -- %ld", u_time);
-    retval = std::string(ftime);
+    retval = fmt::format(FMT_STRING("{} -- {}"), ftime, u_time);
 
     return Ok(retval);
 }
diff --git a/src/third-party/rapidyaml/ryml_all.hpp b/src/third-party/rapidyaml/ryml_all.hpp
index 03734a16..3c1af598 100644
--- a/src/third-party/rapidyaml/ryml_all.hpp
+++ b/src/third-party/rapidyaml/ryml_all.hpp
@@ -498,6 +498,11 @@ C4_FOR_EACH(PRN_STRUCT_OFFSETS, a, b, c);
    #endif
    #define C4_BYTE_ORDER _C4EL
 
+#elif defined(mips) || defined(__mips__) || defined(__mips)
+
+    #define C4_WORDSIZE 8
+    #define C4_BYTE_ORDER _C4EL
+
 #elif defined(__EMSCRIPTEN__)
 #   define C4_BYTE_ORDER _C4EL
 #   define C4_WORDSIZE 4
diff --git a/test/test_cmds.sh b/test/test_cmds.sh
index 5103f79d..edde4027 100644
--- a/test/test_cmds.sh
+++ b/test/test_cmds.sh
@@ -475,6 +475,9 @@ run_cap_test ${lnav_test} -n \
     logfile_rollover.1.live
 
 touch -t 200711030923 ${srcdir}/logfile_syslog.0
+run_cap_test ${lnav_test} -n \
+    ${test_dir}/logfile_syslog.0
+
 run_cap_test ${lnav_test} -n \
     -c ":switch-to-view histogram" \
     -c ":zoom-to 4-hour" \