Gervill update (Was: Bump HotSpot to new HotSpot Express repository and hs14b15)

[Mark Wielaard]

Hi,

On Thu, 2009-05-14 at 22:44 +0200, Mark Wielaard wrote:
> It looks like all of gervill finally
> got merged in b16, I will go through our overlay and compare to see if
> that is why there are 2 failures. And hopefully merge that this weekend.

I ended up just pulling in the new fixes from Gervill CVS into our
overlay. That seemed safer just before the release than changing the
whole build. After the release I'll see about switching completely to
the o6 version.

2009-05-17  Mark Wielaard  <mark at klomp.org>

        * overlays/openjdk/jdk/src/share/classes/com/sun/media/sound/*,
        overlays/openjdk/jdk/test/com/sun/media/sound/*:
        Import Gervill fixes from CVS. See CHANGES.txt:

        6823446: Gervill SoftLowFrequencyOscillator fails when freq is
        set to 0 cent or 8.1758 Hz.
        6823445: Gervill SoftChannel/ResetAllControllers jtreg test
        fails after portamento fix from last merge. 
        6821030: Merge OpenJDK Gervill with upstream sources, Q1CY2009

        - Fix: Notes get stuck on when we overflow the synthesizer
        with rapid note on /off messages.
        JTreg test has been created:
        test/com/sun/media/sound/SoftChannel/NoteOverFlowTest
        - Fix: Portamento controller reset to wrong value in
        SoftChannel.resetAllControllers.
        This caused incorrect glider effect in some midi files.
        For example in TuningApplet1 in TET-7 keyboard,
        when note was pressed for first time a glider effect could be
        heard.
        - Fix: Delay modulation calculation in SoftChorus.LFODelay where
        incorrect.
        - Fix: Removed unnecessary volatile keywords in SoftChorus.

The patch looks bigger than it really is, most of it is whitespace
changes, import cleanups and test updates. The actual changes to the
code are attached.

This makes all sound tests pass, so I would like to see this commit
pulled into the new release if possible.

Thanks,

Mark
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gervill-cvs-update.diff
Type: text/x-patch
Size: 7931 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20090518/989a17e0/gervill-cvs-update.diff