From: Seungha Yang <seungha@centricular.com>
Date: Fri, 6 May 2022 19:43:49 +0000 (+0900)
Subject: tools: device-monitor: Print string property as-is without serialize
X-Git-Tag: 1.22.0~1660
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=66f017f8eacc386a078fe42bf7a14457c2b552a3;p=platform%2Fupstream%2Fgstreamer.git

tools: device-monitor: Print string property as-is without serialize

gst_value_serialize() does more than what's needed to printf-ing
especially when given GValue is already string. Just print string
value as-is without gst_value_serialize() to avoid unreadable
string print, especially for multi-bytes character encoding cases.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2387>
---

diff --git a/subprojects/gst-plugins-base/tools/gst-device-monitor.c b/subprojects/gst-plugins-base/tools/gst-device-monitor.c
index ef57acc..880889e 100644
--- a/subprojects/gst-plugins-base/tools/gst-device-monitor.c
+++ b/subprojects/gst-plugins-base/tools/gst-device-monitor.c
@@ -144,6 +144,8 @@ print_structure_field (GQuark field_id, const GValue * value,
   if (G_VALUE_HOLDS_UINT (value)) {
     val = g_strdup_printf ("%u (0x%08x)", g_value_get_uint (value),
         g_value_get_uint (value));
+  } else if (G_VALUE_HOLDS_STRING (value)) {
+    val = g_value_dup_string (value);
   } else {
     val = gst_value_serialize (value);
   }