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

 
 

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

Submitted by:  Benjamin Wolsey <bwy>
Submitted on:  Sun 22 Jan 2012 11:03:11 AM UTC  
 
Category: NoneSeverity: 4 - Important
Release: NoneStatus: Works For Me
Privacy: PublicAssigned to: None
Open/Closed: Open

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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>
Project 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>
Project 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>
Project Administrator
Sun 22 Jan 2012 11:03:11 AM UTC, original submission:

See attached swf.

Benjamin Wolsey <bwy>
Project Member

 

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

Attach File(s):
   
   
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
  • -unavailable- added by rsavoye (Updated the item)
  • -unavailable- added by bwy (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 23 Jan 2012 06:43:05 PM UTCrsavoyeAssigned torsavoye=>None
    Mon 23 Jan 2012 06:36:20 PM UTCbwyAssigned toNone=>rsavoye
    Sun 22 Jan 2012 06:16:47 PM UTCrsavoyeSeverity5 - Blocker=>4 - Important
      StatusNone=>Works For Me
      Assigned torsavoye=>None
    Sun 22 Jan 2012 11:03:11 AM UTCbwyAttached File-=>Added timer.swf, #24867

    Back to the top


    Powered by Savane 3.1-cleanup