you'll have less overhead (and more portability) if you simply define pi as 3.141659 rather than using Math::Trig.

Of course, that should read 3.14159. Or even 3.14159265358979323846264338327950288419716939937510 :)