From eb329eec1ca2ecc9119e6cee7c338832770d1f72 Mon Sep 17 00:00:00 2001
From: Stefan Sauer <ensonic@users.sf.net>
Date: Mon, 2 Oct 2017 16:22:00 +0200
Subject: [PATCH] structure: add a todo comment

Printing NULL is confusing when the type is e.g. a GArray that is not empty.
---
 gst/gststructure.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/gst/gststructure.c b/gst/gststructure.c
index 8596c2bced..068b4fae8e 100644
--- a/gst/gststructure.c
+++ b/gst/gststructure.c
@@ -1829,6 +1829,7 @@ priv_gst_structure_append_to_gstring (const GstStructure * structure,
         GST_WARNING ("No value transform to serialize field '%s' of type '%s'",
             g_quark_to_string (field->name),
             _priv_gst_value_gtype_to_abbr (type));
+      /* TODO(ensonic): don't print NULL if field->value is not empty */
       g_string_append (s, "NULL");
     }
   }