[gstreamer-bugs] [Bug 359653] GStreamer Binary Registry patch

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=359653

  GStreamer | gstreamer (core) | Ver: HEAD CVS


René Stadler changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mail at renestadler.de




------- Comment #11 from René Stadler  2006-10-08 14:55 UTC -------
Note that Stefan meant pascal *style* strings.  Real pascal strings that are
backed by the compiler don't work anyways since they have a maximum length of
255.

You should most probably change GstBinaryPluginFeature to remove the string
fields and add an array for the string lengths instead.  Directly after writing
out the storage representation of a GstBinaryPluginFeature you write all the
strings of the feature in a defined order, preferably the same order the length
array has.  This will finally give the file a sane size and allow for reading
it back even faster since no scanning for NULL bytes is needed.

The fact that the binary registry is currently much larger than the XML version
should ring all alarm bells; XML is already verbose (read: wasteful, bloated)
because it aims to be human readable.


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email