[FFmpeg-devel] [PATCH]lavf/img2: Allow reading svg image sequences

[Carl Eugen Hoyos]

2017-05-17 15:39 GMT+02:00 Rostislav Pehlivanov <atomnuker at gmail.com>:
> On 17 May 2017 at 13:58, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>
>> 2017-05-17 14:46 GMT+02:00 Rostislav Pehlivanov <atomnuker at gmail.com>:
>> > On 17 May 2017 at 10:03, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
>> >
>> >> Hi!
>> >>
>> >> I believe attached patch is necessary to allow reading svg sequences.
>> >>
>> >> Please comment, Carl Eugen
>>
>> > With this patch svg and svgz files aren't detected as such and
>> > demuxing and decoding fails.
>>
>> Solution would be to adapt the score in svg_probe(), it may be anything
>> but 50 (same score -> no detection).
>>
> So that's why everything in _probe() returns extension score + 1, now
> I get it.
> Could you amend this patch to do that as well and test it? If it works
> feel free to push.

Tested the detection and pushed, thank you!

Should we keep the ff_image_svg_pipe_demuxer?
Or improve the probing by testing for "<!DOCTYPE"?

Carl Eugen