After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 599147 - typefind : Avoid having to re-sort the typefind factory list
typefind : Avoid having to re-sort the typefind factory list
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other Linux
: Normal enhancement
: 0.10.26
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on: 598896
Blocks:
 
 
Reported: 2009-10-21 07:56 UTC by Edward Hervey
Modified: 2009-10-24 08:51 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
typefind: Keep typefind factories sorted in the registry (4.24 KB, patch)
2009-10-21 07:56 UTC, Edward Hervey
accepted-commit_now Details | Review
typefind: Keep typefind factories sorted in the registry (4.49 KB, patch)
2009-10-24 08:15 UTC, Edward Hervey
committed Details | Review

Description Edward Hervey 2009-10-21 07:56:14 UTC
The typefind helper features always re-sort the typefind factory list.

The proposed patch modifies the registry subtlely to returned that factory list pre-sorted (whereas previously the order was undefined).

Depends on #598896
Comment 1 Edward Hervey 2009-10-21 07:56:43 UTC
Created attachment 145925 [details] [review]
typefind: Keep typefind factories sorted in the registry

This avoids having to do the sorting everytime we use typefind

The behaviour of gst_type_find_factory_get_list has subtlely changed
in the sense that the order was previously undefined, whereas now
it returns them sorted by rank and then by name.
Comment 2 Sebastian Dröge (slomo) 2009-10-22 08:46:10 UTC
Review of attachment 145925 [details] [review]:

Looks good... I don't think that behaviour change is a problem. Changing something from being undefined (and different depending on phase of moon and other things) to something defined is ok :)
Comment 3 Edward Hervey 2009-10-24 08:15:07 UTC
Created attachment 146156 [details] [review]
typefind: Keep typefind factories sorted in the registry

This avoids having to do the sorting everytime we use typefind

The behaviour of gst_type_find_factory_get_list has subtlely changed
in the sense that the order was previously undefined, whereas now
it returns them sorted by rank and then by name.
Comment 4 Edward Hervey 2009-10-24 08:51:08 UTC
commit 5067664c09668384d57ccde51e6f2f62e1778cea
Author: Edward Hervey <bilboed@bilboed.com>
Date:   Wed Oct 21 09:48:41 2009 +0200

    typefind: Keep typefind factories sorted in the registry. Fixes #599147
    
    This avoids having to do the sorting everytime we use typefind
    
    The behaviour of gst_type_find_factory_get_list has subtlely changed
    in the sense that the order was previously undefined, whereas now
    it returns them sorted by rank and then by name.