[gstreamer-bugs] [Bug 388050] [neonhttpsrc] code simplifications

[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=388050

  GStreamer | gst-plugins-bad | Ver: HEAD CVS


Tim-Philipp Müller changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #78804|none                        |committed
               Flag|                            |
         AssignedTo|t.i.m at zen.co.uk             |gstreamer-
                   |                            |bugs at lists.sourceforge.net
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
            Summary|[PATCH] simplification of   |[neonhttpsrc] code
                   |the neon plugin             |simplifications
   Target Milestone|HEAD                        |0.10.5




------- Comment #5 from Tim-Philipp Müller  2007-03-02 12:06 UTC -------
Thanks, committed (minus the typefinding bit, for the reasons mentioned above,
ie. mostly 'sources are special'):

  2007-03-02  Tim-Philipp Müller  <tim at centricular dot net>

        Patch by: Lutz Mueller  <lutz topfrose de>

        * ext/neon/gstneonhttpsrc.c: (gst_neonhttp_src_class_init),
        (gst_neonhttp_src_init), (gst_neonhttp_src_set_property),
        (gst_neonhttp_src_set_uri), (gst_neonhttp_src_set_proxy),
        (gst_neonhttp_src_send_request_and_redirect),
        (gst_neonhttp_src_uri_set_uri):
        * ext/neon/gstneonhttpsrc.h:
          Simplify _set_uri() and _set_proxy() and remove the unused ishttp
          member (#388050).

        * tests/check/elements/neonhttpsrc.c: (GST_START_TEST):
          Fix bogus URI to something that actually exists, otherwise we just
          bypass the test (and also to something that doesn't redirect, since
          neonhttpsrc doesn't seem to handle this very gracefully yet)


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