[vlc-commits] mms: remove "http" shortcut

Rémi Denis-Courmont git at videolan.org
Sun Dec 20 16:22:12 CET 2015


vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Sun Dec 20 17:20:31 2015 +0200| [dcae6a70f1176e2fc59a1da64d818e0d91214302] | committer: Rémi Denis-Courmont

mms: remove "http" shortcut

The HTTP module will redirect to MMS(H) when applicable, so handing
of the "http" scheem in the MMS module is no longer necessary.

Removing it saves a pointless connection attempt when HTTP fails for a
reason other than redirecting to MMS.

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=dcae6a70f1176e2fc59a1da64d818e0d91214302
---

 modules/access/mms/mms.c |    5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/modules/access/mms/mms.c b/modules/access/mms/mms.c
index 7f69d61..3584a05 100644
--- a/modules/access/mms/mms.c
+++ b/modules/access/mms/mms.c
@@ -80,7 +80,7 @@ vlc_module_begin ()
     add_string( "mmsh-proxy", NULL, PROXY_TEXT, PROXY_LONGTEXT,
                     false )
 
-    add_shortcut( "mms", "mmsu", "mmst", "mmsh", "http" )
+    add_shortcut( "mms", "mmsu", "mmst", "mmsh" )
     set_callbacks( Open, Close )
 vlc_module_end ()
 
@@ -110,8 +110,7 @@ static int Open( vlc_object_t *p_this )
         {
             return  MMSTUOpen ( p_access );
         }
-        else if( !strncmp( p_access->psz_access, "mmsh", 4 ) ||
-                 !strncmp( p_access->psz_access, "http", 4 ) )
+        else if( !strncmp( p_access->psz_access, "mmsh", 4 ) )
         {
             return  MMSHOpen ( p_access );
         }



More information about the vlc-commits mailing list