[Request for review] 6833357: Improve time-stamp support in Gervill to reduce jitter.
alex.menkov at sun.com
Tue Jul 21 08:36:58 PDT 2009
Looks good for me.
The only note is about something strange with SoftSynthesizer.java (open
method): your patch reapply a fix which is already in the repo.
Looks like you have inconsistent repository.
Karl Helgason wrote:
> I need code review for the fix:
> This fix adds support for zero-jitter playback when using time-stamped MIDI events.
> Full potential of this fix won't come until time-stamp support has been added to RealTimeSequencer.
> see: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6833358
> Here is a overview of changes made:
> SoftLimiter.java - Fix NullPointerException when in mono mode
> SoftChannel.java - Add note on int delay parameter
> SoftReceiver - Add getMidiDevice method and fix midi activity detection
> SoftAudioBuffer - Add swap method
> SoftMainMixer, SoftSynthesizer, SoftVoice - Add delay logic
More information about the jdk6-dev