[vlc-devel] [PATCH] DASH: fix URL parsing

[Frederic YHUEL]

On Wed, Feb 22, 2012 at 12:42 PM, Hugo Beauzée-Luyssen
<beauze.h at gmail.com> wrote:
> On Wed, Feb 22, 2012 at 12:23 PM, Frederic YHUEL <fyhuel at viotech.net> wrote:
>> On Wed, Feb 22, 2012 at 11:44 AM, Hugo Beauzée-Luyssen
>> <beauze.h at gmail.com> wrote:
>>> On Wed, Feb 22, 2012 at 11:42 AM, Frederic YHUEL <fyhuel at viotech.net> wrote:
>>>> 2012/2/22 Christopher Müller <christopher.mueller at itec.uni-klu.ac.at>:
>>>>>
>>>>> My solution should not fix that problem ;) it sets the url of the segments relative to the mpd path in the case when the segment url is not valid. I think you can integrate your solution into mine.
>>>>>
>>>>
>>>> Indeed I read too fast and thought you were just fixing a parsing
>>>> bug... :-)  I'll send a new patch then.
>>>>
>>>> Thanks,
>>>>
>>>
>>> Would for for me too :)
>>> Although, patch does not apply, but since you're about to send a new one... :)
>>>
>>
>> Here is the new one (attached).
>>
>> Best Regards,
>>
>> --
>> Frédéric
>>
>
> Why using std::string::assign instead of the operator= ?
>

Actually "learn C++" is still on my todo list...

Here is a new patch.

Best Regards,

-- 
Frédéric
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-DASH-fix-URL-parsing.patch
Type: text/x-diff
Size: 2917 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20120222/26596d79/attachment.patch>