[kde-edu]: KTurtle patch

[Aleix]

I recommend to use <cmath> over math.h, using math.h can make your
life difficult sometimes... :)

cmath is in the standard since a lot of time ago, we should not use
cmath in the same way that we don't use iostream.h...

Bye!

On 10/3/07, Jeff Shipley <jshipley at jshipley.homelinux.net> wrote:
> Right, it's just a C vs C++ thing.  <math.h> is the C way of doing things,
> and <cmath> is the C++ way.  Either should work on pretty much any C++
> compiler.
>
> --
> Jeff
>
> > Hi Annma,
> >
> > I confirm that <cmath> is the proper name under which c++ programs should
> > include the standard math header. You do raise a good about comptibility
> > with old compilers; I have no clue whether there are problems in that
> > respect; all I know is that I include <cmath> in Eigen and nobody ever
> > reported problems (and Eigen has been used on many different platforms).
> >
> > Cheers,
> > Benoit
> >
> > On Wed, 3 Oct 2007, Anne-Marie Mahfouf wrote:
> >
> >> Le Tuesday 02 October 2007 23:46:47 Jeff Shipley, vous avez écrit:
> >> I have a first patch ready.  The turtle pointed up at 0 degrees and
> >> moved
> >> clockwise as the degrees increased.
> >>
> >> This is different from the way it should work, ie right at 0 degrees and
> >> move counter-clockwise.
> >>
> >> I worked this one out during a lecture at school (I probably should have
> >> been listening more, but...)
> >>
> >> I don't have a kde svn account yet, so I am asking the good people on
> >> this
> >> list to review my patch and check it in for me.
> >>
> >> Thank you,
> >> Jeff
> >
> > Hi Jeff,
> >
> > I reviewed the patch which is OK :)
> > The only thing I windered about was the cmath include but grepping kdelibs
> > I
> > found it used a bit versus math.h so it's OK (I was concerned about old
> > compilers or non-linux systems)
> >
> > I see that you have a svn account so I leave it to you to commit!
> >
> > Anne-Marie
> >
> > _______________________________________________
> > kde-edu mailing list
> > kde-edu at mail.kde.org
> > https://mail.kde.org/mailman/listinfo/kde-edu_______________________________________________
> > kde-edu mailing list
> > kde-edu at mail.kde.org
> > https://mail.kde.org/mailman/listinfo/kde-edu
> >
>
>
> _______________________________________________
> kde-edu mailing list
> kde-edu at mail.kde.org
> https://mail.kde.org/mailman/listinfo/kde-edu
>