structure: add a todo comment
authorStefan Sauer <ensonic@users.sf.net>
Mon, 2 Oct 2017 14:22:00 +0000 (16:22 +0200)
committerStefan Sauer <ensonic@users.sf.net>
Mon, 2 Oct 2017 14:23:31 +0000 (16:23 +0200)
Printing NULL is confusing when the type is e.g. a GArray that is not empty.

gst/gststructure.c

index 8596c2b..068b4fa 100644 (file)
@@ -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");
     }
   }