[FFmpeg-devel] [PATCH] avformat/avio: Add Metacube support

[Derek Buitenhuis]

On 03/05/2021 22:05, Steinar H. Gunderson wrote:
> Two reasons:
> 
>  - As the commit message says, it is desirable to have this on pipe output as
>    well (Cubemap would like to fork out to FFmpeg to do remuxing etc. for
>    it, without having to speak HTTP over the pipe).
>  - The HTTP layer does not have the information about what is a header etc.,
>    so it would need to be plumbed down the layers. I tried this at first, but
>    it got markedly more intrusive than this, so I left it pretty early on.

Can this not be accomplished outside of FFmpeg, by registering your own I/O 
callbacks? That would seem to me to be the 'proper' way to do this. More
work? Yes. But less hacky.

I am in agreement that the current placement inside AVIO itself is king of horrible.

Two other points:
  * Why are there random unexlained changes to e.g. movdec's flushing behavior bundled
    in this patch?
  * You need to provide documentation / links on, like, what 'metacube' *is*. It's
    absolutely unclear to anyone reading this patch and associated doxy what it is
    or what its usecase is. AFAICT it's your own project?

- Derek