[FFmpeg-devel] [PATCH] avutil: Add NV61 pixel format and enable IO support for NV16 and NV61 pixel formats
Cristian Bicheru
c.bicheru0 at gmail.com
Sat Jun 20 08:49:34 EEST 2020
On Fri, Jun 19, 2020 at 6:24 PM Hendrik Leppkes <h.leppkes at gmail.com> wrote:
>
> On Fri, Jun 19, 2020 at 9:58 PM Alexander Strasser <eclipse7 at gmx.net> wrote:
> >
> > How do others think about adding support for more pixel formats?
> >
>
> A new pixel format should present a clear improvement, a use-case you
> couldn't do before, or could only do with a performance penalty or
> whatever.
> If everything that you can do with NV61 you could also do with NV16, a
> format we already have, which value does adding it add to the project?
>
> Completionism should not be a goal. There are hundreds of obscure
> pixel formats that we have no business all adding.
I agree, given the sheer number of these rarely used pixel formats, I
suppose it's not worth the extra code complexity to add them all in.
If we are keen on having support for these pixel formats, I think it
would make more sense to have a way for users to specify custom
formats? But even then I don't know how useful that would be for most
people.
In the meantime, I'll remove the new format from the patch.
More information about the ffmpeg-devel
mailing list