[FFmpeg-devel] [PATCH 0/4] libavcodec/dnxhdenc: add support for dnxhr encoding

[Mark Reid]

On Mon, Jul 18, 2016 at 3:12 PM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> Mark Reid <mindmark <at> gmail.com> writes:
>
>> On Sun, Jul 17, 2016 at 11:06 AM, Carl Eugen Hoyos wrote:
>> > Mark Reid <mindmark <at> gmail.com> writes:
>> >
>> >> The following patch series adds support for dnxhr encoding.
>> >> I added dnxhr as a profile to the dnxhd encoder.
>> >
>> > Is there also a raw format that our demuxer currently does
>> > not auto-detect (and / or not demux)?
>
>> yes, the existing raw dnxhd muxer has support to auto-detect
>> and demux. Would it be a good idea to add a ".dnxhr" extension
>> to that muxer?
>
> To the muxer or the demuxer?
> The muxer is trivial, it is both possible to add a new
> extension or to add a new raw muxer.

I meant add the extension to the raw muxer. Yes it would trival.

> More important (and possibly less trivial) is the demuxer:
> If a new demuxer is needed, imo a new muxer should be added,
> if the existing demuxer can be patched, the existing muxer
> should also be used.
> The question is if the raw format exists in the wild (or
> if we should create it), if it is possible to support it
> in the existing demuxer and if you will create such a patch.
>
> Adding an extension to a demuxer should not be needed, or
> only as a last resort.
>

The raw dnxhd demuxer has support for the format. support was added
couple months ago in commit 8395b6e.
there is also sample and test in fate aswell.
fate/dnxhd/dnxhr_cid1274.dnxhr


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