From b73d4a7022bd1e485f10d66794d35e1cfe78bde9 Mon Sep 17 00:00:00 2001
From: topjohnwu <topjohnwu@gmail.com>
Date: Sun, 16 Feb 2025 11:46:42 -0800
Subject: [PATCH] Fix log_ok()

---
 native/src/base/result.rs | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/native/src/base/result.rs b/native/src/base/result.rs
index 18482e039..c717679d3 100644
--- a/native/src/base/result.rs
+++ b/native/src/base/result.rs
@@ -96,17 +96,6 @@ impl<T, R: Loggable<T>> ResultExt<T> for R {
         self.do_log(LogLevel::Error, Some(Location::caller()))
     }
 
-    #[track_caller]
-    #[cfg(debug_assertions)]
-    fn log_ok(self) {
-        self.log().ok();
-    }
-    
-    #[cfg(not(debug_assertions))]
-    fn log_ok(self) {
-        self.log().ok();
-    }
-
     #[cfg(not(debug_assertions))]
     fn log_with_msg<F: FnOnce(Formatter) -> fmt::Result>(self, f: F) -> LoggedResult<T> {
         self.do_log_msg(LogLevel::Error, None, f)
@@ -117,6 +106,17 @@ impl<T, R: Loggable<T>> ResultExt<T> for R {
     fn log_with_msg<F: FnOnce(Formatter) -> fmt::Result>(self, f: F) -> LoggedResult<T> {
         self.do_log_msg(LogLevel::Error, Some(Location::caller()), f)
     }
+
+    #[cfg(not(debug_assertions))]
+    fn log_ok(self) {
+        self.log().ok();
+    }
+
+    #[track_caller]
+    #[cfg(debug_assertions)]
+    fn log_ok(self) {
+        self.do_log(LogLevel::Error, Some(Location::caller())).ok();
+    }
 }
 
 impl<T> Loggable<T> for LoggedResult<T> {