[FFmpeg-devel] [PATCH] url_split() ipv6 support

Michael Niedermayer michaelni
Fri Sep 28 17:04:40 CEST 2007


Hi

On Fri, Sep 28, 2007 at 12:33:04AM -0400, Ronald S. Bultje wrote:
> Hi,
> 
> On 9/27/07, Andreas ?man <andreas at olebyn.nu> wrote:
> >
> > Applied
> 
> 
> Then here's #2 for ipv6 support in address parsing.
> 
> $ ./test-url_split
> "test.avi"-> "" :// "" @ "" : "-1" / "test.avi"
> "/tmp/test.mp3"-> "" :// "" @ "" : "-1" / "/tmp/test.mp3"
> "file:///tmp/test.mp3"-> "file" :// "" @ "" : "-1" / "/tmp/test.mp3"
> "http://www.address.com"-> "http" :// "" @ "www.address.com" : "-1" / ""
> "http://www.address.com/"-> "http" :// "" @ "www.address.com" : "-1" / "/"
> "http://www.address.com:80"-> "http" :// "" @ "www.address.com" : "80" / ""
> "http://www.address.com:80/"-> "http" :// "" @ "www.address.com" : "80" /
> "/"
> "http://www.address.com:80/test.mp3"-> "http" :// "" @ "www.address.com" :
> "80" / "/test.mp3"
> "http://1.2.3.4:80/test.mp4"-> "http" :// "" @ "1.2.3.4" : "80" /
> "/test.mp4"
> "http://:1::80:1/test.mov"-> "http" :// "" @ ":1::80:1" : "-1" / "/test.mov"
> "http://:::80/test.mp5"-> "http" :// "" @ "::" : "80" / "/test.mp5"
> "http://[::1]:80/test.mp6"-> "http" :// "" @ "::1" : "80" / "/test.mp6"
> "http://[host.address]:80/test.mp3"-> "http" :// "" @ "host.address" : "80"
> / "/test.mp3"
> "file:///[pat]h/to/file.mp3"-> "file" :// "" @ "" : "-1" /
> "/[pat]h/to/file.mp3"
> "http://1:2:3:4:5:6:7:8:80/test.mp3"-> "http" :// "" @ "1:2:3:4:5:6:7:8" :
> "80" / "/test.mp3"
> "http://me@:::80/test.mp3"-> "http" :// "me" @ "::" : "80" / "/test.mp3"
> "http://me:x@:::80/test.mp3"-> "http" :// "me:x" @ "::" : "80" / "/test.mp3"
> 
> All are good now.

but this is significantly too complex for my taste
why dont you just support the notation from rfc2732

where does the other notation came from anyway? if its not in a rfc
its not correct and i see no sense in supporting it that is unless
it can be done naturaly with no increase in complexity ...

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Thouse who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070928/29c5a666/attachment.pgp>



More information about the ffmpeg-devel mailing list