[FFmpeg-devel] [PATCH v3 2/2] lavf: add prores bitstream demuxer and muxer

[Andreas Rheinhardt]

hung kuishing:
>> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of
>> Derek Buitenhuis
>> Sent: Wednesday, July 26, 2023 8:55 PM
>> To: ffmpeg-devel at ffmpeg.org
>> Subject: Re: [FFmpeg-devel] [PATCH v3 2/2] lavf: add prores bitstream
>> demuxer and muxer
>>
>> On 7/26/2023 10:28 AM, hung kuishing wrote:
>>> Signed-off-by: clarkh <hungkuishing at outlook.com>
>>> ---
>>>  libavformat/Makefile     |  2 ++
>>>  libavformat/allformats.c |  2 ++
>>>  libavformat/proresdec.c  | 66
>> ++++++++++++++++++++++++++++++++++++++++
>>>  libavformat/rawenc.c     | 13 ++++++++
>>>  4 files changed, 83 insertions(+)
>>>  create mode 100644 libavformat/proresdec.c
>>
>> At this point I am giving this a strong NAK.
>>
>> Both my initial comment[1] and subsequent comment[2] about the
>> first one being ignore, have been ignored. It is a simple question.
> 
> Sorry for not replying to your question in time!
> This patch originated from a need in my work to wrap the ProRes bitstream generated by ffmpeg into another mov wrapper.

So there are no files in the wild for this? Then there is no point in
this. Or is this something that other mov wrappers (you meant muxers!?)
accept?
(Couldn't you just have used e.g. nut (or even mov itself) to
temporarily store the ProRes data and then pipe this to to the foreign
muxer?)

- Andreas