:: Re: [Frei0r] 4 new plugins announce…
Góra strony
Delete this message
Reply to this message
Autor: Marko Cebokli
Data:  
Dla: Minimalistic plugin API for video effects
Temat: Re: [Frei0r] 4 new plugins announcement
Wouldn't it be simpler to just prescribe the use of byte pointers?
The speed penalty on modern CPUs is probably minimal, especially as most of
them are now 64 bit, and 32bit accesses can be unaligned too - and cache lines
are even bigger than 64.

Marko Cebokli



On Monday 11 June 2012 01:50:05 salsaman wrote:
> Yes, Solaris is tricky because it can run on Sparc (bigendian) or PC
> (littleendian) I think.
> Maybe fall back to the runtime check in that case.
>
>
> Salsaman.
>
>
>
> http://lives.sourceforge.net
> https://www.ohloh.net/accounts/salsaman
>
> On Sun, Jun 10, 2012 at 2:38 PM, Dan Dennedy <dan@???> wrote:
> > On Sun, Jun 10, 2012 at 8:47 AM, salsaman <salsaman@???> wrote:
> >> Hi Jaromil,
> >> maybe its a good time to look at the endianness issue - at least add
> >> the #defines into frei0r.h.
> >
> > I modified your previous proposal to include more well-defined OS
> > detection based on this:
> > https://sourceforge.net/apps/mediawiki/predef/index.php?title=Operating_S
> > ystems
> >
> > #ifndef _WIN32
> > #ifndef sun
> > #include <endian.h>
> > #define F0R_BYTE_ORDER __BYTE_ORDER
> > #define F0R_LITTLE_ENDIAN __LITTLE_ENDIAN
> > #define F0R_BIG_ENDIAN __BIG_ENDIAN
> >
> > #else // Solaris or SunOS
> > #endif
> >
> > #else // Windows
> > #define F0R_LITTLE_ENDIAN 1
> > #define F0R_BIG_ENDIAN 0
> > #define F0R_BYTE_ORDER F0R_LITTLE_ENDIAN
> > #endif
> >
> > --
> > Note I changed the IS_ prefix to a F0R_ with a zero to be consistent
> > with other defines in the header. What should we do about sun?
> >
> > There is interesting discussion on stackoverflow:
> > http://stackoverflow.com/questions/2100331/c-macro-definition-to-determin
> > e-big-endian-or-little-endian-machine
> >
> > A much later comment suggested this, which looks elegant:
> >
> > #define F0R_LITTLE_ENDIAN (*(uint16_t*)"\0\1">>8)
> > #define F0R_BIG_ENDIAN (*(uint16_t*)"\1\0">>8)
> >
> > However, they are macros and not for the pre-processor so when used
> > within a loop will degrade performance.
> >
> >> Salsaman.
> >>
> >> http://lives.sourceforge.net
> >> https://www.ohloh.net/accounts/salsaman
> >>
> >> On Sun, Jun 10, 2012 at 6:10 AM, Jaromil <jaromil@???> wrote:
> >>> On Fri, 08 Jun 2012, Dan Dennedy wrote:
> >>>> I pushed Janne's new plugins to dyne.org.
> >>>
> >>> thanks Dan!
> >>>
> >>> I will add your github account to the information on the website ASAP
> >>> (eheh! now its you on github!!!) so that those who want to contribute
> >>> using that can hopefully do the same way.
> >>>
> >>> And what about rounding up a new release? its about the time i think,
> >>> or do we see any important pending issues to fix/settle? I can
> >>> dedicate some hours to frei0r in the coming week so let me know
> >>>
> >>> ciao
> >>>
> >>> --
> >>> jaromil, dyne.org developer, http://jaromil.dyne.org
> >>> GPG: B2D9 9376 BFB2 60B7 601F 5B62 F6D3 FBD9 C2B6 8E39
> >>>
> >>>
> >>>
> >>> _______________________________________________
> >>> Frei0r mailing list - http://frei0r.dyne.org
> >>> Free video plugins, minimal and cross-platform.
> >>> http://mailinglists.dyne.org/cgi-bin/mailman/listinfo/frei0r
> >>
> >> _______________________________________________
> >> Frei0r mailing list - http://frei0r.dyne.org
> >> Free video plugins, minimal and cross-platform.
> >> http://mailinglists.dyne.org/cgi-bin/mailman/listinfo/frei0r
> >
> > --
> > +-DRD-+
> > _______________________________________________
> > Frei0r mailing list - http://frei0r.dyne.org
> > Free video plugins, minimal and cross-platform.
> > http://mailinglists.dyne.org/cgi-bin/mailman/listinfo/frei0r
>
> _______________________________________________
> Frei0r mailing list - http://frei0r.dyne.org
> Free video plugins, minimal and cross-platform.
> http://mailinglists.dyne.org/cgi-bin/mailman/listinfo/frei0r