Patch for audi/midi services <was> Re: Audio/Midi services <was> Fwd: CR 7172234 Created, P2 java/jigsaw Lot of sound test failed due UnsatisfiedLinkError for com.sun.media.sound.* classes

[Paul Sandoz]

Hi,

See end of email for a patch to add the audi/midi services.

Here is some sampled output from dump-config for the desktop context (proof that is it working):

    local service providers (14)
      javax.sound.midi.spi.MidiFileWriter (1)
        com.sun.media.sound.StandardMidiFileWriter

      javax.sound.midi.spi.MidiFileReader (1)
        com.sun.media.sound.StandardMidiFileReader

      javax.sound.sampled.spi.MixerProvider (2)
        com.sun.media.sound.PortMixerProvider
        com.sun.media.sound.DirectAudioDeviceProvider

      javax.sound.sampled.spi.AudioFileReader (5)
        com.sun.media.sound.AuFileReader
        com.sun.media.sound.AiffFileReader
        com.sun.media.sound.WaveFileReader
        com.sun.media.sound.WaveFloatFileReader
        com.sun.media.sound.SoftMidiAudioFileReader

      javax.sound.sampled.spi.FormatConversionProvider (4)
        com.sun.media.sound.AudioFloatFormatConverter
        com.sun.media.sound.UlawCodec
        com.sun.media.sound.AlawCodec
        com.sun.media.sound.PCMtoPCMCodec

      javax.sound.sampled.spi.AudioFileWriter (3)
        com.sun.media.sound.AuFileWriter
        com.sun.media.sound.AiffFileWriter
        com.sun.media.sound.WaveFileWriter

      javax.sound.midi.spi.MidiDeviceProvider (4)
        com.sun.media.sound.RealTimeSequencerProvider
        com.sun.media.sound.MidiOutDeviceProvider
        com.sun.media.sound.MidiInDeviceProvider
        com.sun.media.sound.SoftProvider

      javax.sound.midi.spi.SoundbankReader (4)
        com.sun.media.sound.SF2SoundbankReader
        com.sun.media.sound.DLSSoundbankReader
        com.sun.media.sound.AudioFileSoundbankReader
        com.sun.media.sound.JARSoundbankReader


It seems the pattern in jdk.depconfig is to always use optional, so i kept it like that, but since there are always services for the audi/midi cases, we don't need it to be optional, correct?

Paul.

$ hg diff
diff -r 68564a2f4750 make/modules/jdk.depconfig
--- a/make/modules/jdk.depconfig	Wed May 30 13:00:52 2012 +0100
+++ b/make/modules/jdk.depconfig	Wed May 30 16:36:43 2012 +0200
@@ -542,3 +542,27 @@
 @Provider(optional)
   javax.xml.soap.SOAPFactory -> META-INF/services/javax.xml.soap.SOAPFactory
 
+ at Provider(optional)
+  javax.sound.midi.spi.MidiDeviceProvider -> META-INF/services/javax.sound.midi.spi.MidiDeviceProvider
+
+ at Provider(optional)
+  javax.sound.midi.spi.MidiFileReader -> META-INF/services/javax.sound.midi.spi.MidiFileReader
+
+ at Provider(optional)
+  javax.sound.midi.spi.MidiFileWriter -> META-INF/services/javax.sound.midi.spi.MidiFileWriter
+
+ at Provider(optional)
+  javax.sound.midi.spi.SoundbankReader -> META-INF/services/javax.sound.midi.spi.SoundbankReader
+
+ at Provider(optional)
+  javax.sound.sampled.spi.AudioFileReader -> META-INF/services/javax.sound.sampled.spi.AudioFileReader
+
+ at Provider(optional)
+  javax.sound.sampled.spi.AudioFileWriter -> META-INF/services/javax.sound.sampled.spi.AudioFileWriter
+
+ at Provider(optional)
+  javax.sound.sampled.spi.FormatConversionProvider -> META-INF/services/javax.sound.sampled.spi.FormatConversionProvider
+
+ at Provider(optional)
+  javax.sound.sampled.spi.MixerProvider -> META-INF/services/javax.sound.sampled.spi.MixerProvider
+