[FFmpeg-devel] [PATCH 4/5] avutil/pixdesc: add AV_PIX_FMT_FLAG_ALPHA to AV_PIX_FMT_PAL8

Michael Niedermayer michael at niedermayer.cc
Wed Apr 25 20:44:52 EEST 2018


On Tue, Apr 24, 2018 at 09:05:00PM +0200, Marton Balint wrote:
> Signed-off-by: Marton Balint <cus at passwd.hu>
> ---
>  doc/APIchanges                | 3 +++
>  libavcodec/tests/imgconvert.c | 4 ----
>  libavutil/pixdesc.c           | 3 +--
>  libavutil/pixdesc.h           | 8 ++------
>  libavutil/tests/pixdesc.c     | 4 ----
>  libavutil/version.h           | 2 +-
>  6 files changed, 7 insertions(+), 17 deletions(-)

this with the rest of the patchset seems not to break anything
so no objections from me

i was wondering though if when a 2nd PAL8 is introduced which will
be with alpha

PAL8 and PAL8A seemed a natural choice name wise
iam mentioning this as if these 2 would be used then the addition of alpha
to PAL8 would have to be undone.
so it would make sense to first decide if the new format will be with or 
without alpha

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180425/89361842/attachment.sig>


More information about the ffmpeg-devel mailing list