[FFmpeg-devel] [PATCH] Prefer "monowhite"/"monoblack" over "monow"/"monob"
Stefano Sabatini
stefano.sabatini-lala
Thu Apr 23 23:52:01 CEST 2009
On date Wednesday 2009-04-01 02:12:44 +0200, Stefano Sabatini encoded:
> On date Wednesday 2009-04-01 02:02:35 +0200, Michael Niedermayer encoded:
> > On Wed, Apr 01, 2009 at 01:47:29AM +0200, Stefano Sabatini wrote:
> > > Hi,
> > >
> > > the names are more explicative and consistent with the corresponding
> > > pixel formats macros.
> >
> > iam against this chnage, its also called rgb abd not redgreenblue
>
> But:
>
> PIX_FMT_ARGB -> "argb"
> PIX_FMT_MONOWHITE -> "monow"
>
> "monow" and "monob" are the only exceptions to the pixel format
> macro/name correspondence.
Sorry to bother again for this, I was almost committing the pixdescs
definition, I noted you used "monoblack" for the PIX_FMT_MONOBLACK
pixdesc, this maybe means that your subconsious agrees with me that
there should be a *strict* correspondance between PIX_FMT_ macros and
names, or we should fix that.
BTW the same PIX_FMT_/"name" inconsistency also happens for other
formats, follows the complete list:
PIX_FMT_MONOWHITE -> "monow"
PIX_FMT_MONOBLACK -> "monob"
PIX_FMT_GRAY8 -> "gray"
PIX_FMT_XVMC_MPEG2_MC -> "xvmcmc"
PIX_FMT_XVMC_MPEG2_IDCT -> "xvmcidct"
So the question is: should we keep this in pixdescs too or should we
normalize PIX_FMT_ macros and names?
I already pedantically expressed my position.
Regards.
--
FFmpeg = Friendly Faithful Mastodontic Peaceless Elastic Genius
More information about the ffmpeg-devel
mailing list