[vlc-devel] [PATCH 5a/5] stream_ReadLine: control maximum length with object variable

Rémi Denis-Courmont remi at remlab.net
Sun Sep 6 08:47:43 CEST 2020


Le sunnuntaina 6. syyskuuta 2020, 5.46.36 EEST Pierre Ynard via vlc-devel a 
écrit :
> Very long lines are occasionally encountered in text web resources such
> as HTML, JSON or other API data, which the current hardcoded limit can't
> accommodate. This allows the caller to configure that limit to do so.
> 
> The default remains unchanged at 200 kB.

Which is already insanely high. For comparison, Kate limits lines at 4,096 
characters by default...

There are no valid use cases for doing this in Lua. You have not exhibited any 
line-based file formats that needs larger lines. As François and I already 
reminded you multiples times, HTML is NOT line-based.

-- 
雷米‧德尼-库尔蒙
http://www.remlab.net/





More information about the vlc-devel mailing list