[PATCH v2 17/42] XFree86: sdksyms: Remove unused -DXorgLoader

Peter Hutterer peter.hutterer at who-t.net
Tue Dec 6 15:48:46 PST 2011


On Fri, Dec 02, 2011 at 11:27:25AM +0000, Daniel Stone wrote:
> We no longer have anything in the tree that checks for XorgLoader.  This
> was a fairly monumental hack: xvdi.h used to hide all its functions
> behind #ifndef XorgLoader, solely to avoid sdksyms.sh picking up its
> symbols, as it was previously a module rather than built-in.
> 
> This is no longer the case, so we can remove the define.
> 
> Signed-off-by: Daniel Stone <daniel at fooishbar.org>
> ---

Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>

> 
> v2: Split out from Xv patch.
> 
>  hw/xfree86/sdksyms.sh |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/hw/xfree86/sdksyms.sh b/hw/xfree86/sdksyms.sh
> index b8563ee..38115ba 100755
> --- a/hw/xfree86/sdksyms.sh
> +++ b/hw/xfree86/sdksyms.sh
> @@ -321,7 +321,7 @@ topdir=$1
>  shift
>  LC_ALL=C
>  export LC_ALL
> -${CPP:-cpp} "$@" -DXorgLoader sdksyms.c | ${AWK:-awk} -v topdir=$topdir '
> +${CPP:-cpp} "$@" sdksyms.c | ${AWK:-awk} -v topdir=$topdir '
>  BEGIN {
>      sdk = 0;
>      print("/*");
> -- 
> 1.7.7.3
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
> 


More information about the xorg-devel mailing list