[gstreamer-bugs] [Bug 333779] New: API: add gst_uri_has_protocol

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

           Summary: API: add gst_uri_has_protocol
           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: t.i.m at zen.co.uk
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


I'd like to add the following API:

  gboolean  gst_uri_has_protocol (const gchar * uri, const gchar * protocol);


Almost all GstUriHandler implementations have overly complicated code blocks
like

  gchar *proto;

  proto = gst_uri_get_protocol (uri);
  if (!proto || strcmp (proto, "myproto") != 0) {
    g_free (proto);
    return FALSE;
  }

  g_free (proto);


Would be much nicer to have just a simple

  if (!gst_uri_has_protocol (uri, "myproto"))
    return FASLE;


-- 
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.