[FFmpeg-devel] [PATCH 1/3] Add muxing/demuxing of RMHD

Carl Eugen Hoyos ceffmpeg at gmail.com
Thu Jan 18 22:04:04 EET 2018


2018-01-18 19:39 GMT+01:00 Thilo Borgmann <thilo.borgmann at 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


More information about the ffmpeg-devel mailing list