Re: [AD] Header splitup.

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


> I think the point was to allow a platform to define asm version of
> some routines, and have fallback versions.

This works for platforms that have asm versions of the routines and
explicitly disable them by defining ALLEGRO_NO_ASM very early. Now for
platforms that don't have asm versions, asm.inl is clever enough to
automatically define ALLEGRO_NO_ASM. The new layout breaks this mechanism.

> If putting everything in the (platform independant) .inl files, that
> will mean it contains a lot of #ifdef THAT_PLATFORM stuff, which is
> not very nice...

No. If you take a look at asm.inl, you'll see that it is only a wrapper for
the platform-dependent asm file, which are located in the platform/
directory.

IMHO gfxasm.inl and mathasm.inl are useless because you can't include them
without including *each time* gfx.inl and fmaths.inl (respectively) before.
This oversight broke the compilation under BC++.

--
Eric Botcazou
ebotcazou@xxxxxxxxxx



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