bugGnash - The GNU Flash player - Bugs: bug #35351, Regression: timers broken when...

 
 

bug #35351: Regression: timers broken when framerate above 12

Submitter:  Benjamin Wolsey <bwy>
Submitted:  Sun 22 Jan 2012 11:03:11 AM UTC
   
 
Category:  None Severity:  4 - Important
Release:  None Status:  Works For Me
Privacy:  Public Assigned to:  None
Open/Closed:  Open
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 23 Jan 2012 06:43:05 PM UTC, comment #3: 

You missed the point as well, but I've been looking at improving it anyway.

Rob Savoye <rsavoye>
Group administrator
Mon 23 Jan 2012 06:36:20 PM UTC, comment #2: 

You appear to have missed the point. It is not about the exact count, but about the resolution. You'll see that the Flash player now outpaces Gnash by about 4 to 1.

Framerate can vary depending on external factors like CPU load. Timers use the system time. You've broken it so that a framerate of 13 makes the minimum timer resolution 76ms instead of the expected 10. A function that's supposed to be called every 20ms will only be called approximately every 80ms.

Also this was a testcase I produced in 5 minutes showing just one of the side-effects of the change. I could write any number of others showing different problems. Please fix.

Benjamin Wolsey <bwy>
Group Member
Sun 22 Jan 2012 06:16:47 PM UTC, comment #1: 

I still get falsecounts even with the delay set to 10ms. Setting "set delay 10" in the gnashrc file overrides any of the FPS based changes, so this is easy to test. I'm more interested in real world testing for the release. End users prefer a lower CPU load.

Rob Savoye <rsavoye>
Group administrator
Sun 22 Jan 2012 11:03:11 AM UTC, original submission:  

See attached swf.

Benjamin Wolsey <bwy>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #24867:  timer.swf added by bwy (227B - application/x-shockwave-flash)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rsavoye (Updated the item)
  • -email is unavailable- added by bwy (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2012-01-23 rsavoye Assigned torsavoye None
    2012-01-23 bwy Assigned toNone rsavoye
    2012-01-22 rsavoye Severity5 - Blocker 4 - Important
        StatusNone Works For Me
        Assigned torsavoye None
    2012-01-22 bwy Attached File- Added timer.swf, #24867

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code