[gstreamer-bugs] [Bug 350432] New: micro-optimising quarks

[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=350432
 GStreamer | gstreamer (core) | Ver: HEAD CVS

           Summary: micro-optimising quarks
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gstreamer (core)
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: thaytan at mad.scientist.com
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


Attaching a patch which adds a new non-installed header gstquark.h, and some
new private data that allows us to pre-register some static strings with
g_quark_from_static_string and save looking up quarks when inserting values
into GstStructures.

As a first place to use it, all the structure accesses in GstQuery now go
straight from stored quarks. The same can be done for events, and the same
technique could be used in helper functions in libgstvideo/audio to avoid Quark
lookups.

It's a small optimisation, but it made a degenerate test case of
create-each-query,parse-each-query about 10 times faster.


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.