On Mon, Jul 18, 2016 at 3:12 PM, Carl Eugen Hoyos <ceho...@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@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
