Re: [FFmpeg-devel] [PATCH v3 1/2] avformat/url: rework for trim_double_dot_url

2020-07-27 Thread Zlomek, Josef
Generally, this seems to be too complicated with too much of copying. On Mon, Jul 27, 2020 at 3:08 PM Steven Liu wrote: > use two buffer for storage the path and node of URI > remove the last node of the path if the next node is .. > change the static strings to dynamic alloc space by size argum

Re: [FFmpeg-devel] [PATCH v3 1/2] avformat/url: rework for trim_double_dot_url

2020-07-27 Thread Nicolas George
Steven Liu (12020-07-27): > use two buffer for storage the path and node of URI > remove the last node of the path if the next node is .. > change the static strings to dynamic alloc space by size argument. ?!?!??!!!? Why do you think you need extra buffers? This patch turns a rather straightfor

[FFmpeg-devel] [PATCH v3 1/2] avformat/url: rework for trim_double_dot_url

2020-07-27 Thread Steven Liu
use two buffer for storage the path and node of URI remove the last node of the path if the next node is .. change the static strings to dynamic alloc space by size argument. Signed-off-by: Steven Liu --- libavformat/url.c | 83 --- 1 file changed, 64