[Bug 781438] New: value: NULL string serialization/deserialization

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=781438

            Bug ID: 781438
           Summary: value: NULL string serialization/deserialization
    Classification: Platform
           Product: GStreamer
           Version: unspecified
                OS: Linux
            Status: NEW
          Severity: blocker
          Priority: Normal
         Component: gstreamer (core)
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: slomo at coaxion.net
        QA Contact: gstreamer-bugs at lists.freedesktop.org
                CC: olivier.crete at ocrete.ca
     GNOME version: ---

Following commits:
http://cgit.freedesktop.org/gstreamer/gstreamer/commit/?id=0b9eb621b69b06df350ba25cb3a07ad9d23aef88
http://cgit.freedesktop.org/gstreamer/gstreamer/commit/?id=7699b8a1bc49f30cd9a0c300831a085516f5442e
http://cgit.freedesktop.org/gstreamer/gstreamer/commit/?id=dd1f0f49abfb35e38e7a2fa3a9229a384fbd0c9b

I think we disallow NULL strings for a reason, and in taglists they are
explicitly forbidden. What is your use-case for these changes, why do you need
NULL strings? Can't you use "" or just not set the string at all? Arguably NULL
is not a valid string at all.

IMHO we should revert these for 1.12 and reconsider that afterwards. It's not a
change that should be added right before the release as we will have to support
this behaviour forever.


Same goes for GstStructure (3rd commit).

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.