From: Fritzsche K. <k.f...@dk...> - 2008-09-24 15:52:22
|
Hi, everything fine now. Thank you! Regards, Klaus -----Ursprüngliche Nachricht----- Von: Gordon Kindlmann [mailto:gk...@bw...] Gesendet: Mittwoch, 24. September 2008 16:29 An: Fritzsche Klaus Cc: Teem Users Betreff: Re: AW: [Teem-users] Windows compile hi- Right- okay, I've committed those changes now; make sure "svn update" gives you something that works. Gordon On Sep 24, 2008, at 10:04 AM, Fritzsche Klaus wrote: > Hi, > > your changes are resolving most of the issues. I had to change the > M_PI constants to AIR_PI to be able to compile under windows. > > Best regards, > Klaus > > > Index: src/ten/tenGage.c > =================================================================== > --- src/ten/tenGage.c (revision 3893) > +++ src/ten/tenGage.c (working copy) > @@ -1586,13 +1586,13 @@ > /* make sure we use the minimal rotation */ > isoPhi=asin(2.0/A*(mean-0.5*(submatrix[0]+submatrix[2]))); > if (isoPhi>0) { > - if (fabs(M_PI-isoPhi-C)<fabs(isoPhi-C)) > - isoPhi=0.5*(M_PI-isoPhi-C); > + if (fabs(AIR_PI-isoPhi-C)<fabs(isoPhi-C)) > + isoPhi=0.5*(AIR_PI-isoPhi-C); > else > isoPhi=0.5*(isoPhi-C); > } else if (isoPhi<0) { > - if (fabs(-M_PI-isoPhi-C)<fabs(isoPhi-C)) > - isoPhi=0.5*(-M_PI-isoPhi-C); > + if (fabs(-AIR_PI-isoPhi-C)<fabs(isoPhi-C)) > + isoPhi=0.5*(-AIR_PI-isoPhi-C); > else > isoPhi=0.5*(isoPhi-C); > } > > -----Ursprüngliche Nachricht----- > Von: Gordon Kindlmann [mailto:gk...@bw...] > Gesendet: Mittwoch, 24. September 2008 15:45 > An: James Bigler > Cc: Fritzsche Klaus; Teem Users > Betreff: Re: [Teem-users] Windows compile > > Hi- > > Its a GNU cc compiler flag, and I'd use it more often except that > Teem has always relied on the "long long" type, and so far its worked > everywhere, but "long long" isn't in C89, so its annoying to compile > it with "-std=c89 -pedantic" since every source file gives that > working. There's probably a way to turn that one warning off, but > its also sufficient to just occasionally compile with that (certainly > before any release) and make sure that there aren't C++isms > > Gordon > > On Sep 24, 2008, at 9:32 AM, James Bigler wrote: > >> Is this compiler flag specific to a particular compiler? This flag >> should probably be used for those compilers that don't do this by >> default. >> >> James >> >> On Wed, Sep 24, 2008 at 7:29 AM, Gordon Kindlmann >> <gk...@bw...> wrote: >>> hello, >>> >>> I also apologize for not seeing those problems- now that I've >>> used "- >>> std=c89 -pedantic" I've found and I believe fixed all these >>> problems. >>> >>> Try doing another svn update and see if things compile okay now. >>> >>> Gordon >>> >>> On Sep 24, 2008, at 6:33 AM, Thomas Schultz wrote: >>> >>>> Hi, >>>> >>>> that was my code, then, sorry. I am used to writing C++ and find it >>>> hard >>>> to not even use features of C99 in my code (that standard is now >>>> nine >>>> years old and current Windows compilers do not support it? Oh, >>>> well...). >>>> I will try and stick to "classic" C in future contributions. >>>> >>>> Best /Thomas >>>> >>>> ------------------------------------------------------------------- >>>> - >>>> -- >>>> --- >>>> This SF.Net email is sponsored by the Moblin Your Move Developer's >>>> challenge >>>> Build the coolest Linux based applications with Moblin SDK & win >>>> great prizes >>>> Grand prize is a trip for two to an Open Source event anywhere in >>>> the world >>>> http://moblin-contest.org/redirect.php?banner_id=100&url=/ >>>> _______________________________________________ >>>> teem-users mailing list >>>> tee...@li... >>>> https://lists.sourceforge.net/lists/listinfo/teem-users >>> >>> >>> -------------------------------------------------------------------- >>> - >>> ---- >>> This SF.Net email is sponsored by the Moblin Your Move Developer's >>> challenge >>> Build the coolest Linux based applications with Moblin SDK & win >>> great prizes >>> Grand prize is a trip for two to an Open Source event anywhere in >>> the world >>> http://moblin-contest.org/redirect.php?banner_id=100&url=/ >>> _______________________________________________ >>> teem-users mailing list >>> tee...@li... >>> https://lists.sourceforge.net/lists/listinfo/teem-users >>> > > <tenGage.diff> |