[FFmpeg-devel] [PATCH] Create a libavutil/pix_fmt.h with the pixel format stuff

Michael Niedermayer michaelni
Fri Feb 20 21:08:55 CET 2009


On Thu, Feb 19, 2009 at 11:29:45PM +0100, Stefano Sabatini wrote:
> On date Thursday 2009-02-19 22:00:59 +0100, Stefano Sabatini encoded:
> > On date Thursday 2009-02-19 13:57:24 +0100, Michael Niedermayer encoded:
> > > On Thu, Feb 19, 2009 at 01:19:22AM +0100, Stefano Sabatini wrote:
> > > > On date Thursday 2009-02-19 00:20:23 +0100, Michael Niedermayer encoded:
> > > > > On Thu, Feb 19, 2009 at 12:03:26AM +0100, Stefano Sabatini wrote:
> > > > > > On date Sunday 2009-02-15 20:58:35 +0100, Vitor Sessak encoded:
> > > > > > > Stefano Sabatini wrote:
> > > > > > > > Hi,
> > > > > > > > 
> > > > > > > > the algorithm implemented in draw_slice() only works with planar
> > > > > > > > formats, it may be extended and it will but for the moment this patch
> > > > > > > > fixes filtering for filter chains of the kind:
> > > > > > > > "format=rgb32, hflip"
> > > > > > > 
> > > > > > > I think it shouldn't be too hard to extend it, but I'm fine with the patch.
> > > > > > > 
> > > > > > > > Also what about a function like avfilter_all_planar_colorspaces()?
> > > > > > > 
> > > > > > > 
> > > > > > > Maybe something like
> > > > > > > 
> > > > > > > avfilter_colorspaces(int flags);
> > > > > > > 
> > > > > > > enum flags = {
> > > > > > >      FMT_IS_PLANAR = 1;
> > > > > > >      FMT_IS_YUV = 2;
> > > > > > >      FMT_IS_RGB = 4;
> > > > > > >      FMT_IS_REAL_PIXELS = 8; ///< Not a HW format
> > > > > > >      FMT_IS_BW = 16; ///< Is it useful?
> > > > > > > }
> > > > > > > would be more useful? 
> > > > > > 
> > > > > > I like your idea. Now I see there are many possibly useful macros
> > > > > > (isPlanarYUV, isYUV, isGray, isGray16, etc.) in swscale_internal.h,
> > > > > > the alternatives are to move them to lavu, or to duplicate the code in
> > > > > > lavfi.
> > > > > > 
> > > > > > My idea was to move the macros to lavu, redefine them something like:
> > > > > > AV_PIX_FMT_IS_PLANAR_YUV()
> > > > > > AV_PIX_FMT_IS_YUV()
> > > > > > AV_PIX_FMT_IS_GRAY()
> > > > > > ...
> > > > > > 
> > > > > > and redefine the current libsws macros like:
> > > > > > #define isPlanarYUV(x) AV_PIX_FMT_IS_PLANAR_YUV(x)
> > > > > > 
> > > > > > swscale_internal.h already depends on libavutil/avutil.h.
> > > > > > 
> > > > > > Would be this a acceptable?
> > > > > 
> > > > > first the macros in sws are inefficiently implemented,
> > > > > an array so that
> > > > > 
> > > > > AV_PIX_FMT_IS_YUV(x) (array[x]&0x01)
> > > > > would be a good idea, also see pix_fmt_info
> > > > 
> > > > Yes, I'm thinking about to move PixFmtInfo to lavu too.
> > > >  
> > > > > if its well and clean implemenetd lavu is an option also sws should n that
> > > > > case use lavus variant directly and not use wraper macros
> > > > 
> > > > First step creates a pix_fmt.h header (PixFmtInfo would then be added
> > > > to libavutil/pix_fmt.c)
> > > 
> > > PixFmtInfo as is is too bloated, it requires cleanup _first_
> > > also your patch misses installing the new header while a installed header
> > > depends on it
> > 
> > Yes.
> > 
> > Also I'm not sure if pixfmt.h (no underscore) is a better name.
> 
> Uh, patch missing...

probably ok


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

Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090220/300d5172/attachment.pgp>



More information about the ffmpeg-devel mailing list