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

[Carl Eugen Hoyos]

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.
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.

Carl Eugen