On Thu, Jan 3, 2019 at 2:32 AM Cedric Bail <[email protected]> wrote:
>
> On Wednesday, January 2, 2019 1:24 PM, Vincent Torri 
> <[email protected]> wrote:
> > just for some ideas, here is a patch for this code :
> > https://pastebin.com/bHdMVjLZ
>
> It would be really nice to start pushing this through phab for
> review as this would allow for a better tracking of it for sure. I
> really recommend switching to git phab as described:
> https://phab.enlightenment.org/w/arcanist/ (I know the page title
> is "slightly" misleading).

i will use these functions in efl code too, then

> > I have to add a macro (EINA_BIGENDIAN) with configure as the code is inlined
> >
> > ideas, remarks ?
>
> I think you should use the macro WORDS_BIGENDIAN which is already
> available in both meson and the autotools.

no, they are **inlined** functions that is, if a library or program
**uses** these functions, WORDS_BIGENDIAN may be not correctly
defined.

I can use WORDS BIGENDIAN if the functions are not inlined

> I think I kind of understand why you wouldn't want to use them even
> on Linux and I am fine the way it looks and seems to generate the
> proper asm at the end.

thank you

Vincent


_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to