glib: Use actual function name for structured logging and also log source file/line number

Author: sdroege

glib/src/log.rs

@@ -859,36 +859,57 @@ macro_rules! g_printerr {
 /// ```
 #[macro_export]
 macro_rules! log_structured {
-    ($log_domain:expr, $log_level:expr, {$($key:expr => $format:expr $(,$arg:expr)* $(,)?);+ $(;)?} $(,)?) => {
-        (|| {
-            let log_domain = <Option<&str> as std::convert::From<_>>::from($log_domain);
-            let log_domain_str = log_domain.unwrap_or_default();
-            let level: $crate::LogLevel = $log_level;
-            let field_count =
-                <[()]>::len(&[$($crate::log_structured_inner!(@clear $key)),+])
-                + log_domain.map(|_| 2usize).unwrap_or(1usize);
-
-            $crate::log_structured_array(
-                level,
-                &[
-                    $crate::LogField::new(
-                        $crate::gstr!("PRIORITY"),
-                        level.priority().as_bytes(),
-                    ),
-                    $(
-                        $crate::LogField::new(
-                            $crate::log_structured_inner!(@key $key),
-                            $crate::log_structured_inner!(@value $format $(,$arg)*),
-                        ),
-                    )+
+    ($log_domain:expr, $log_level:expr, {$($key:expr => $format:expr $(,$arg:expr)* $(,)?);+ $(;)?} $(,)?) => {{
+        let log_domain = <Option<&str> as std::convert::From<_>>::from($log_domain);
+        let log_domain_str = log_domain.unwrap_or_default();
+        let level: $crate::LogLevel = $log_level;
+        let field_count =
+            <[()]>::len(&[$($crate::log_structured_inner!(@clear $key)),+])
+            + log_domain.map(|_| 2usize).unwrap_or(1usize)
+            + 3;
+
+        let mut line = [0u8; 32]; // 32 decimal digits of line numbers should be enough!
+        let line = {
+            use std::io::Write;
+
+            let mut cursor = std::io::Cursor::new(&mut line[..]);
+            std::write!(&mut cursor, "{}", line!()).unwrap();
+            let pos = cursor.position() as usize;
+            &line[..pos]
+        };
+
+        $crate::log_structured_array(
+            level,
+            &[
+                $crate::LogField::new(
+                    $crate::gstr!("PRIORITY"),
+                    level.priority().as_bytes(),
+                ),
+                $crate::LogField::new(
+                    $crate::gstr!("CODE_FILE"),
+                    file!().as_bytes(),
+                ),
+                $crate::LogField::new(
+                    $crate::gstr!("CODE_LINE"),
+                    line,
+                ),
+                $crate::LogField::new(
+                    $crate::gstr!("CODE_FUNC"),
+                    $crate::function_name!().as_bytes(),
+                ),
+                $(
                     $crate::LogField::new(
-                        $crate::gstr!("GLIB_DOMAIN"),
-                        log_domain_str.as_bytes(),
+                        $crate::log_structured_inner!(@key $key),
+                        $crate::log_structured_inner!(@value $format $(,$arg)*),
                     ),
-                ][0..field_count],
-            )
-        })()
-    };
+                )+
+                $crate::LogField::new(
+                    $crate::gstr!("GLIB_DOMAIN"),
+                    log_domain_str.as_bytes(),
+                ),
+            ][0..field_count],
+        )
+    }};
 }
 
 #[doc(hidden)]

glib/tests/structured_log.rs

@@ -82,12 +82,22 @@ fn structured_log() {
             )
         })
         .collect::<Vec<_>>();
+
+    let path = if cfg!(windows) {
+        "glib\\tests\\structured_log.rs"
+    } else {
+        "glib/tests/structured_log.rs"
+    };
+
     assert_eq!(
         log[0],
         (
             LogLevel::Message,
             vec![
                 ("PRIORITY", "5" as &str),
+                ("CODE_FILE", path as &str),
+                ("CODE_LINE", "30" as &str),
+                ("CODE_FUNC", "structured_log::structured_log" as &str),
                 ("MY_META", "abc"),
                 ("MESSAGE", "normal with meta"),
                 ("MY_META2", "def"),
@@ -101,6 +111,9 @@ fn structured_log() {
             LogLevel::Message,
             vec![
                 ("PRIORITY", "5" as &str),
+                ("CODE_FILE", path as &str),
+                ("CODE_LINE", "40" as &str),
+                ("CODE_FUNC", "structured_log::structured_log" as &str),
                 ("MY_META", "abc"),
                 ("MESSAGE", "formatted with meta: 123 456"),
                 ("MY_META2", "defghi"),