On 15/10/2012 07:43, Yaakov (Cygwin/X) wrote:
> From: Yaakov Selkowitz
> <yselkowitz-rn4veauk+akrv+lv9mx5uipxlwaov...@public.gmane.org>
>
> These functions are already declared in <X11/fonts/fontproto.h>.
> Redeclaring them just for _X_EXPORT causes tons of warnings throughout
> xserver which can be avoided by handling this in the function
> definitions instead.
>
> Signed-off-by: Yaakov Selkowitz
> <yselkowitz-rn4veauk+akrv+lv9mx5uipxlwaov...@public.gmane.org>
> ---
>
> dix/dixfonts.c | 9 +++++++++
> include/dixfont.h | 38 --------------------------------------
> 2 files changed, 9 insertions(+), 38 deletions(-)
>
> diff --git a/dix/dixfonts.c b/dix/dixfonts.c
> index ad21860..feb765d 100644
> --- a/dix/dixfonts.c
> +++ b/dix/dixfonts.c
> @@ -1823,12 +1823,14 @@ InitFonts(void)
> register_fpe_functions();
> }
>
> +_X_EXPORT
> int
> GetDefaultPointSize(void)
> {
> return 120;
> }
I also had a go at fixing this annoying warning (see [1]), but I don't think
this is quite straightforward, as sdksyms.sh only looks at the headers to
build the list of symbols which must be referenced.
[1] http://lists.x.org/archives/xorg-devel/2012-January/028887.html
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
