POV-Ray: Newsgroups: povray.programming: Two possible minor bugs in pvengine.c: Two possible minor bugs in pvengine.c

POV-Ray : Newsgroups : povray.programming : Two possible minor bugs in pvengine.c : Two possible minor bugs in pvengine.c		Server Time 3 Jul 2024 07:15:57 EDT (-0400)

From: Severi Salminen
Date: 4 Feb 2004 02:55:02
Message: <4020a556$1@news.povray.org>

If found next possible (very minor) bugs in pvengine.c

Line 665:

if (ClockTimeTotal >= 1000)

Should probably be:

if (ClockTimeTotal >= CLOCKS_PER_SEC)

This makes it more consistent on different platforms. And if CLOCKS_PER_SEC 
is defined very low, it might get a while to see the actual PPS reading :)


Line 4294:

status_printf (StatusPPS, PPS_String (pixels, ClockTimeTotal / 1000)) ;

Should probably be:

status_printf (StatusPPS, PPS_String (pixels, ClockTimeTotal / 
CLOCKS_PER_SEC)) ;

These are, of course, very minor and affect barely anything. On non-Windows 
platforms (or non Visual C++ compilers?) the PPS reading might be wrong 
when rendering is paused, if CLOCKS_PER_SEC is not 1000. Including the fix 
to 3.6 would be probably very safe - if needed to.

Severi Salminen

Post a reply to this message