2018-01-18 19:39 GMT+01:00 Thilo Borgmann <thilo.borgm...@mail.de>:
> This time including the patch...

Why do you add a new demuxer?
Isn't it sufficient to add the new descriptor (with AV_RB32()) to the
existing probe function?

Please split this patch:
From a (very) quick look, I saw:
Demuxing MLTI, muxing aac, muxing RV30 and friends, some
fixes wrt version (all apart from RV60).
Please add a few comments where it currently says "unknown".
I suspect no new muxer should be added (but I may miss something).

Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to