Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.2.6

Bug #342 Redundant Sleep
Submitted: 2003-12-03 14:05 UTC
From: ctcnet at ctco dot lv Assigned: farell
Status: Closed Package: HTML_Progress
PHP Version: 4.3.3 OS: any
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 48 + 45 = ?

 
 [2003-12-03 14:05 UTC] ctcnet at ctco dot lv
Description: ------------ After Progress bar is updated it have sleep(1) statement. This makes execution to pause for 1 second. If progress bar updates 10 times during the script execution, the total time of script execution increases by 10 seconds. This feature may be very annoying/problematic. If progress bar is used in time-critical application, the sleep statement must be commented out. At lease sleep time could be configurable through parameter. I could provide the patch, just need to learn how to properly check out che latest version then diff it to my changes. Not sure if my patch will be acceprted though. If nobody read this or nobody want to fix it, i will provide the patch eventually.

Comments

 [2003-12-03 21:56 UTC] farell
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better. The php-function sleep() is no more necessary on API v1.0