[FFmpeg-devel] [FFmpeg-cvslog] lavf/url: rewrite ff_make_absolute_url() using ff_url_decompose().

Nicolas George george at nsup.org
Sat Aug 22 13:59:56 EEST 2020


Alexander Strasser (12020-08-15):
> If I'm not mistaken that reason for the rewrite didn't make
> it into the commit message:

You are right. The reason for the complete rewrite was that the original
code was completely broken which I did not want to include permanently
in the history.

By completely broken, I mean it used redundant string searches without
bothering about delimiters and syntax. That mean some parts would
parse "/redirect?url=http://example.com/" with everything until "://"
considered the protocol.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200822/e2f4445b/attachment.sig>


More information about the ffmpeg-devel mailing list