[haiku-commits] Re: r40109 - haiku/trunk/src/add-ons/screen_savers/butterfly

  • From: Ryan Leavengood <leavengood@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 4 Jan 2011 12:45:53 -0500

On 1/4/11, Ingo Weinhold <ingo_weinhold@xxxxxx> wrote:
> Moreover, please note that replacing e.g. 15.0f (or 15.f) by 15.0 forces
> the computation to double precision. So, unless that's desired, please
> leave the 15.0f or, preferrably, just use 15, which is int and gets
> automatically promoted to float when necessary.

OK I'll change some back or use ints.

> I have serious trouble seeing the purpose of this computation (why not
> simply 1.f / 6?) or the comment for that matter.

Please keep in mind I was just adding someone else's code and decided
to not do major changes or refactorings since it seemed to work as is.
And this is just grunt work which I decided to do to help Scott in the
GCI. But I see your point.


Other related posts: