Re: [AD] Request: adding textout_ex(), which takes bg color as parameter

[ Thread Index | Date Index | More lists.liballeg.org/allegro-developers Archives ]


> This patch breaks the build process for me, because fsel.c still uses
> the moved OLD_FILSESEL_* defines (lines 761,771).

Sorry, I didn't test the compilation under Linux.

> Index: src/fsel.c
> ===================================================================
> RCS file: /cvsroot/alleg/allegro/src/fsel.c,v
> retrieving revision 1.29
> diff -u -r1.29 fsel.c
> --- src/fsel.c 1 Oct 2002 17:31:46 -0000 1.29
> +++ src/fsel.c 1 Oct 2002 18:33:06 -0000
> @@ -31,6 +31,7 @@
>
>
>  #include "allegro.h"
> +#include "allegro/alcompat.h"
>  #include "allegro/internal/aintern.h"

I don't know. According to these lines in allegro.h:

#ifndef ALLEGRO_LIB_BUILD
   #include "allegro/alcompat.h"
#endif

it seems it is not safe to include alcompat.h while building the library on
Unix. I'm not competent to approve here.

As my patch was posted to the [AD] list, it is easy to locally revert. So I
propose to wait for Peter's opinion until tomorrow.

--
Eric Botcazou



Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/