[libav-devel] libavutil/pixfmt.h and other libavutil weirdness

Måns Rullgård mans at mansr.com
Sat Nov 19 13:06:11 CET 2011


Luca Barbato <lu_zero at gentoo.org> writes:

> I'm trying to make our doxy a little more rational, I started with
> libavutil since it supposedly would be simpler
>
> I just started to add some groups so some poor soul willing to figure
> out what is available gets some proper aggregate.
>
> here some initial questions
>
> libavutil/avutil.h
>> #define FF_LAMBDA_SHIFT 7
>> #define FF_LAMBDA_SCALE (1<<FF_LAMBDA_SHIFT)
>> #define FF_QP2LAMBDA 118 ///< factor to convert from H.263 QP to lambda
>> #define FF_LAMBDA_MAX (256*128-1)
>>
>> #define FF_QUALITY_SCALE FF_LAMBDA_SCALE //FIXME maybe remove
>
> Why those encoding only flags are here? Can't they moved around?

Those macros obviously belong in lavc.  Please move them.

> libavutil/pixfmt.h
>  * @warning This file has to be considered an internal but installed, *
> header, so it should not be directly included in your projects.

Delete that nonsense.

-- 
Måns Rullgård
mans at mansr.com


More information about the libav-devel mailing list