2018-01-18 19:39 GMT+01:00 Thilo Borgmann <[email protected]>:
> 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
[email protected]
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to