Re: Request for review: 7196576: javac warnings cleanup from Adopt OpenJDK bugathon

[Alex Menkov]

Hi Martijn,

Generally the fix looks good, but there are some notes:
src/share/classes/com/sun/media/sound/AlawCodec.java

  // set the AudioInputStream length in frames if we know it
  if (stream instanceof AudioInputStream) {
-     frameLength = ((AudioInputStream)stream).getFrameLength();
+     frameLength = stream.getFrameLength();
  }

stream is always instance of AudioInputStream and is not null
so the check for instanceof could be dropped:

- // set the AudioInputStream length in frames if we know it
- if (stream instanceof AudioInputStream) {
-     frameLength = ((AudioInputStream)stream).getFrameLength();
- }
+ frameLength = stream.getFrameLength();


src/share/classes/com/sun/media/sound/UlawCodec.java
 - the same note as for AlawCodec.java


src/share/classes/javax/sound/midi/MidiSystem.java:

  } catch (MidiUnavailableException e) {
      // something went wrong with synth
      if (e instanceof MidiUnavailableException) {
-         mue = (MidiUnavailableException) e;
+         mue = e;
      }
  }

analogically should be
  } catch (MidiUnavailableException e) {
      // something went wrong with synth
-      if (e instanceof MidiUnavailableException) {
-         mue = (MidiUnavailableException) e;
-      }
+      mue = e;
  }

and a bit more complex case several line later:
  // then try to connect to the default Receiver
  try {
      rec = MidiSystem.getReceiver();
  } catch (Exception e) {
      // something went wrong. Nothing to do then!
      if (e instanceof MidiUnavailableException) {
          mue = (MidiUnavailableException) e;
      }
  }

MidiSystem.getReceiver() may throw only MidiUnavailableException, so the 
statements could be simplified to:
  // then try to connect to the default Receiver
  try {
      rec = MidiSystem.getReceiver();
  } catch (MidiUnavailableException e) {
      // something went wrong. Nothing to do then!
      mue = e;
  }


regards
Alex

On 15.09.2012 16:10, Martijn Verburg wrote:
Hi all,

Artem Ananiev very kindly raised bugs and a webrev for the patches
sent in from a Bugathon we ran back in April (patches have been tested
against latest source tree).

The bug numbers are:

7196576: [Bugathon] Reduce the number of javac warnings in JavaSound

The corresponding webrevs are at:

http://cr.openjdk.java.net/~art/Bugathon-2012/webrev.media/

Thanks to Artem, Stuart and Phil for helping out with the correct
splitting of patches etc.

Cheers,
Martijn