Merge pull request #293 from elfenpiff/iox2-292-fix-log-macros

[#292] Fix log macro bug
eclipse-iceoryx · Jul 14, 2024 · 1354b2b · 1354b2b
2 parents 22357f5 + fa75000
commit 1354b2b
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/doc/release-notes/iceoryx2-unreleased.md b/doc/release-notes/iceoryx2-unreleased.md
@@ -66,6 +66,7 @@
  * Build failure for Windows 11 i686-pc-windows-msvc [#235](https://github.com/eclipse-iceoryx/iceoryx2/issues/235)
  * 'win32call' needs to provide the last error [#241](https://github.com/eclipse-iceoryx/iceoryx2/issues/241)
  * Mem-leak in `iceoryx2-bb-posix::Directory::contents()` and skip empty file names [#287](https://github.com/eclipse-iceoryx/iceoryx2/issues/287)
+ * Log macros do no longer return values [#292](https://github.com/eclipse-iceoryx/iceoryx2/issues/292)
 
 ### Refactoring
 

diff --git a/iceoryx2-bb/log/src/log.rs b/iceoryx2-bb/log/src/log.rs
@@ -49,7 +49,6 @@ macro_rules! trace {
             if result.is_err() {
                 $crate::__internal_print_log_msg($crate::LogLevel::Trace, std::format_args!("{:?}", $o), std::format_args!($($e),*))
             }
-            result.ok().unwrap()
         }
     }
 }
@@ -91,7 +90,6 @@ macro_rules! debug {
             if result.is_err() {
                 $crate::__internal_print_log_msg($crate::LogLevel::Debug, std::format_args!("{:?}", $o), std::format_args!($($e),*))
             }
-            result.ok().unwrap()
         }
     }
 }
@@ -133,7 +131,6 @@ macro_rules! info {
             if result.is_err() {
                 $crate::__internal_print_log_msg($crate::LogLevel::Info, std::format_args!("{:?}", $o), std::format_args!($($e),*))
             }
-            result.ok().unwrap()
         }
     }
 }
@@ -175,7 +172,6 @@ macro_rules! warn {
             if result.is_err() {
                 $crate::__internal_print_log_msg($crate::LogLevel::Warn, std::format_args!("{:?}", $o), std::format_args!($($e),*))
             }
-            result.ok().unwrap()
         }
     }
 }
@@ -217,7 +213,6 @@ macro_rules! error {
             if result.is_err() {
                 $crate::__internal_print_log_msg($crate::LogLevel::Error, std::format_args!("{:?}", $o), std::format_args!($($e),*))
             }
-            result.ok().unwrap()
         }
     }
 }