[PATCH 2/6] xfree86: Report Present as a built-in module
walter harms
wharms at bfs.de
Wed May 28 01:50:20 PDT 2014
Am 28.05.2014 09:14, schrieb Chris Wilson:
> This is so that drivers can do a runtime check that Present is available,
> similar to existing runtime checks performed by the drivers for DRI.
>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> ---
> hw/xfree86/loader/loadmod.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/hw/xfree86/loader/loadmod.c b/hw/xfree86/loader/loadmod.c
> index f92ad8d..e1f649a 100644
> --- a/hw/xfree86/loader/loadmod.c
> +++ b/hw/xfree86/loader/loadmod.c
> @@ -841,6 +841,9 @@ static const char *compiled_in_modules[] = {
> #if DRI3
> "dri3",
> #endif
> +#if PRESENT
> + "present",
> +#endif
> NULL
> };
>
just a remark,
PRESENT is not very descriptive (if DRI3 is better is a second point)
I would suggest a strategy like HAVE_PRESENT.
As non-english native it is easier to read.
and a second one:
I have seen sich #if forest grow in other projects, would it be an option
to have something like _PRESENT_TXT and more that somewhere else ?
#if PRESENT
#define _PRESENT_TXT "present",
#else
#define _PRESENT_TXT
#endif
Of cause, such files can be generate if a pattern is agreed.
just my 2 cents,
wh
More information about the xorg-devel
mailing list