[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