[vlc-devel] [PATCH 1/2] vlc_url_t: Add a b_ipv6 field

[Hugo Beauzée-Luyssen]

On 05/23/2016 02:02 PM, Rémi Denis-Courmont wrote:
> Le 2016-05-23 13:59, Hugo Beauzée-Luyssen a écrit :
>> ---
>>  include/vlc_url.h | 1 +
>>  src/text/url.c    | 2 ++
>>  2 files changed, 3 insertions(+)
>
> There are no "is this IPv6" field on URL schemes. In most cases, it is
> either meaningless or unspecified anyway.
>
> So no thanks.
>

The goal here is to avoid losing the information provided by the square 
brackets.
My specific use case being that the UPnP sees a few UPnP shares that are 
exposing an IPV6 endpoint. We then parse this URL using vlc_UrlParse, 
but psz_host gets stripped of its square brackets. When we use the 
parsed URL to generate another UPnP URL, we don't have the brackets 
anymore, which confuses the accesses into thinking part of the address 
is a port.
I guess we could simply strstr(url.psz_host, ":"), but that seems fairly 
cumbersome & repetitive.