Approved: [Request for review] 6823449: Gervill: ArrayIndexOutOfBoundsException thrown when trying to play too may voices at same time.

[Alex Menkov]

approval for the fix for 6823449

Regards
Alex

Karl Helgason wrote:
> Hi,
> 
> I need code review for the fix:
> http://cr.openjdk.java.net/~kalli/6823449/webrev.01/
> 
> And I need somebody to change the bug description:
> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6823449
> 
> Also I need new bug ids for these bugs/improvements:
> 
> [New Bug]: Improve time-stamp support in Gervill to reduce jitter.
> [New Bug]: Add support for sending time-stamp information in RealTimeSequencer to reduce jitter in playback.
> 
> Those two bugs reduces playback glitches when playing back midi files like described here:
>   https://gervill.dev.java.net/servlets/ProjectForumMessageView?forumID=2923&messageID=26785
> 
> And also add new bug id for this bug:
> 
> [New Bug]: Gervill: Add support to limit maximum CPU load by dynamically limit maximum voice polyphony.
> 
> regards,
> Karl