Re: [AD] proposed patch for nicer asm symbols exports

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


On Thu, Aug 05, 2004, Evert Glebbeek wrote:
> > I think I'm going to apply the following modified patch instead:
> 
> Committed.

    Thanks. Unfortunately I missed a minor issue. Can you please commit
the attached patch for consistency? PREFIX is always undefined under
Unix, but it just feels safer.

-- 
Sam.
Index: src/i386/asmdef.c
===================================================================
RCS file: /cvsroot/alleg/allegro/src/i386/asmdef.c,v
retrieving revision 1.20
diff -u -r1.20 asmdef.c
--- src/i386/asmdef.c	5 Aug 2004 08:17:35 -0000	1.20
+++ src/i386/asmdef.c	5 Aug 2004 15:29:22 -0000
@@ -236,7 +236,7 @@
 #ifdef ALLEGRO_WATCOM
   {"#define FUNC(name)            .globl " PREFIX "name ; nop ; _align_ ; " PREFIX "name:", 0},
 #elif defined ALLEGRO_UNIX
-  {"#define FUNC(name)            .globl " PREFIX "name ; _align_ ; .type name,@function ; " PREFIX "name:", 0},
+  {"#define FUNC(name)            .globl " PREFIX "name ; _align_ ; .type " PREFIX "name,@function ; " PREFIX "name:", 0},
 #else
   {"#define FUNC(name)            .globl " PREFIX "name ; _align_ ; " PREFIX "name:", 0},
 #endif


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