[FFmpeg-devel] Add support for 's' type gopher URL

Submitted by François Revol on April 24, 2017, 4:19 p.m.

Details

Message ID 20170424161913.12285-1-revol@free.fr
State New
Headers show

Commit Message

François Revol April 24, 2017, 4:19 p.m.
It's not standard but mentionned on:
http://en.wikipedia.org/wiki/Gopher_%28protocol%29#Gopher_item_types

It's used at least on:
gopher://sdf.org/1/sdf/historical
(commercial.mp3)
---
 libavformat/gopher.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Michael Niedermayer April 25, 2017, 2:05 a.m.
On Mon, Apr 24, 2017 at 06:19:13PM +0200, François Revol wrote:
> It's not standard but mentionned on:
> http://en.wikipedia.org/wiki/Gopher_%28protocol%29#Gopher_item_types
> 
> It's used at least on:
> gopher://sdf.org/1/sdf/historical
> (commercial.mp3)
> ---
>  libavformat/gopher.c | 1 +
>  1 file changed, 1 insertion(+)

LGTM

thx

[...]

Patch hide | download patch | download mbox

diff --git a/libavformat/gopher.c b/libavformat/gopher.c
index 3070b24caf..7d27d6febe 100644
--- a/libavformat/gopher.c
+++ b/libavformat/gopher.c
@@ -46,6 +46,7 @@  static int gopher_connect(URLContext *h, const char *path)
     switch (*++path) {
         case '5':
         case '9':
+        case 's': /* unofficial type for audio */
             path = strchr(path, '/');
             if (!path) return AVERROR(EINVAL);
             break;