Checkstyle left curly.

Author: Gary Gregory

checkstyle.xml

@@ -38,6 +38,7 @@ limitations under the License.
     <module name="RedundantImport"/>
     <module name="UnusedImports"/>
     <module name="NeedBraces"/>
+    <module name="LeftCurly"/>
     <module name="JavadocMethod">
     </module>
  </module>

src/main/java/org/apache/commons/io/EndianUtils.java

@@ -272,11 +272,9 @@ public static double readSwappedDouble(final byte[] data, final int offset) {
      * @param value value to write
      * @throws IOException in case of an I/O problem
      */
-    public static void writeSwappedShort(final OutputStream output, final short value)
-        throws IOException
-    {
-        output.write( (byte)( ( value >> 0 ) & 0xff ) );
-        output.write( (byte)( ( value >> 8 ) & 0xff ) );
+    public static void writeSwappedShort(final OutputStream output, final short value) throws IOException {
+        output.write((byte) ((value >> 0) & 0xff));
+        output.write((byte) ((value >> 8) & 0xff));
     }
 
     /**
@@ -286,11 +284,8 @@ public static void writeSwappedShort(final OutputStream output, final short valu
      * @return the value just read
      * @throws IOException in case of an I/O problem
      */
-    public static short readSwappedShort(final InputStream input)
-        throws IOException
-    {
-        return (short)( ( ( read( input ) & 0xff ) << 0 ) +
-            ( ( read( input ) & 0xff ) << 8 ) );
+    public static short readSwappedShort(final InputStream input) throws IOException {
+        return (short) (((read(input) & 0xff) << 0) + ((read(input) & 0xff) << 8));
     }
 
     /**
@@ -300,30 +295,25 @@ public static short readSwappedShort(final InputStream input)
      * @return the value just read
      * @throws IOException in case of an I/O problem
      */
-    public static int readSwappedUnsignedShort(final InputStream input)
-        throws IOException
-    {
-        final int value1 = read( input );
-        final int value2 = read( input );
-
-        return ( ( ( value1 & 0xff ) << 0 ) +
-            ( ( value2 & 0xff ) << 8 ) );
+    public static int readSwappedUnsignedShort(final InputStream input) throws IOException {
+        final int value1 = read(input);
+        final int value2 = read(input);
+
+        return (((value1 & 0xff) << 0) + ((value2 & 0xff) << 8));
     }
 
     /**
-     * Writes a "int" value to an OutputStream. The value is
-     * converted to the opposed endian system while writing.
+     * Writes a "int" value to an OutputStream. The value is converted to the opposed endian system while writing.
+     * 
      * @param output target OutputStream
      * @param value value to write
      * @throws IOException in case of an I/O problem
      */
-    public static void writeSwappedInteger(final OutputStream output, final int value)
-        throws IOException
-    {
-        output.write( (byte)( ( value >> 0 ) & 0xff ) );
-        output.write( (byte)( ( value >> 8 ) & 0xff ) );
-        output.write( (byte)( ( value >> 16 ) & 0xff ) );
-        output.write( (byte)( ( value >> 24 ) & 0xff ) );
+    public static void writeSwappedInteger(final OutputStream output, final int value) throws IOException {
+        output.write((byte) ((value >> 0) & 0xff));
+        output.write((byte) ((value >> 8) & 0xff));
+        output.write((byte) ((value >> 16) & 0xff));
+        output.write((byte) ((value >> 24) & 0xff));
     }
 
     /**
@@ -333,18 +323,13 @@ public static void writeSwappedInteger(final OutputStream output, final int valu
      * @return the value just read
      * @throws IOException in case of an I/O problem
      */
-    public static int readSwappedInteger(final InputStream input)
-        throws IOException
-    {
-        final int value1 = read( input );
-        final int value2 = read( input );
-        final int value3 = read( input );
-        final int value4 = read( input );
-
-        return ( ( value1 & 0xff ) << 0 ) +
-            ( ( value2 & 0xff ) << 8 ) +
-            ( ( value3 & 0xff ) << 16 ) +
-            ( ( value4 & 0xff ) << 24 );
+    public static int readSwappedInteger(final InputStream input) throws IOException {
+        final int value1 = read(input);
+        final int value2 = read(input);
+        final int value3 = read(input);
+        final int value4 = read(input);
+
+        return ((value1 & 0xff) << 0) + ((value2 & 0xff) << 8) + ((value3 & 0xff) << 16) + ((value4 & 0xff) << 24);
     }
 
     /**
@@ -354,17 +339,13 @@ public static int readSwappedInteger(final InputStream input)
      * @return the value just read
      * @throws IOException in case of an I/O problem
      */
-    public static long readSwappedUnsignedInteger(final InputStream input)
-        throws IOException
-    {
-        final int value1 = read( input );
-        final int value2 = read( input );
-        final int value3 = read( input );
-        final int value4 = read( input );
-
-        final long low = ( ( ( value1 & 0xff ) << 0 ) +
-                     ( ( value2 & 0xff ) << 8 ) +
-                     ( ( value3 & 0xff ) << 16 ) );
+    public static long readSwappedUnsignedInteger(final InputStream input) throws IOException {
+        final int value1 = read(input);
+        final int value2 = read(input);
+        final int value3 = read(input);
+        final int value4 = read(input);
+
+        final long low = (((value1 & 0xff) << 0) + ((value2 & 0xff) << 8) + ((value3 & 0xff) << 16));
 
         final long high = value4 & 0xff;
 
@@ -378,17 +359,15 @@ public static long readSwappedUnsignedInteger(final InputStream input)
      * @param value value to write
      * @throws IOException in case of an I/O problem
      */
-    public static void writeSwappedLong(final OutputStream output, final long value)
-        throws IOException
-    {
-        output.write( (byte)( ( value >> 0 ) & 0xff ) );
-        output.write( (byte)( ( value >> 8 ) & 0xff ) );
-        output.write( (byte)( ( value >> 16 ) & 0xff ) );
-        output.write( (byte)( ( value >> 24 ) & 0xff ) );
-        output.write( (byte)( ( value >> 32 ) & 0xff ) );
-        output.write( (byte)( ( value >> 40 ) & 0xff ) );
-        output.write( (byte)( ( value >> 48 ) & 0xff ) );
-        output.write( (byte)( ( value >> 56 ) & 0xff ) );
+    public static void writeSwappedLong(final OutputStream output, final long value) throws IOException {
+        output.write((byte) ((value >> 0) & 0xff));
+        output.write((byte) ((value >> 8) & 0xff));
+        output.write((byte) ((value >> 16) & 0xff));
+        output.write((byte) ((value >> 24) & 0xff));
+        output.write((byte) ((value >> 32) & 0xff));
+        output.write((byte) ((value >> 40) & 0xff));
+        output.write((byte) ((value >> 48) & 0xff));
+        output.write((byte) ((value >> 56) & 0xff));
     }
 
     /**
@@ -398,14 +377,12 @@ public static void writeSwappedLong(final OutputStream output, final long value)
      * @return the value just read
      * @throws IOException in case of an I/O problem
      */
-    public static long readSwappedLong(final InputStream input)
-        throws IOException
-    {
+    public static long readSwappedLong(final InputStream input) throws IOException {
         final byte[] bytes = new byte[8];
-        for ( int i=0; i<8; i++ ) {
-            bytes[i] = (byte) read( input );
+        for (int i = 0; i < 8; i++) {
+            bytes[i] = (byte) read(input);
         }
-        return readSwappedLong( bytes, 0 );
+        return readSwappedLong(bytes, 0);
     }
 
     /**
@@ -415,10 +392,8 @@ public static long readSwappedLong(final InputStream input)
      * @param value value to write
      * @throws IOException in case of an I/O problem
      */
-    public static void writeSwappedFloat(final OutputStream output, final float value)
-        throws IOException
-    {
-        writeSwappedInteger( output, Float.floatToIntBits( value ) );
+    public static void writeSwappedFloat(final OutputStream output, final float value) throws IOException {
+        writeSwappedInteger(output, Float.floatToIntBits(value));
     }
 
     /**
@@ -428,10 +403,8 @@ public static void writeSwappedFloat(final OutputStream output, final float valu
      * @return the value just read
      * @throws IOException in case of an I/O problem
      */
-    public static float readSwappedFloat(final InputStream input)
-        throws IOException
-    {
-        return Float.intBitsToFloat( readSwappedInteger( input ) );
+    public static float readSwappedFloat(final InputStream input) throws IOException {
+        return Float.intBitsToFloat(readSwappedInteger(input));
     }
 
     /**
@@ -441,10 +414,8 @@ public static float readSwappedFloat(final InputStream input)
      * @param value value to write
      * @throws IOException in case of an I/O problem
      */
-    public static void writeSwappedDouble(final OutputStream output, final double value)
-        throws IOException
-    {
-        writeSwappedLong( output, Double.doubleToLongBits( value ) );
+    public static void writeSwappedDouble(final OutputStream output, final double value) throws IOException {
+        writeSwappedLong(output, Double.doubleToLongBits(value));
     }
 
     /**
@@ -454,10 +425,8 @@ public static void writeSwappedDouble(final OutputStream output, final double va
      * @return the value just read
      * @throws IOException in case of an I/O problem
      */
-    public static double readSwappedDouble(final InputStream input)
-        throws IOException
-    {
-        return Double.longBitsToDouble( readSwappedLong( input ) );
+    public static double readSwappedDouble(final InputStream input) throws IOException {
+        return Double.longBitsToDouble(readSwappedLong(input));
     }
 
     /**
@@ -466,13 +435,11 @@ public static double readSwappedDouble(final InputStream input)
      * @return the byte
      * @throws IOException if the end of file is reached
      */
-    private static int read(final InputStream input)
-        throws IOException
-    {
+    private static int read(final InputStream input) throws IOException {
         final int value = input.read();
 
-        if( EOF == value ) {
-            throw new EOFException( "Unexpected EOF reached" );
+        if (EOF == value) {
+            throw new EOFException("Unexpected EOF reached");
         }
 
         return value;

src/main/java/org/apache/commons/io/input/ObservableInputStream.java

@@ -92,7 +92,9 @@ public void closed() throws IOException {
          * @param pException the exception to throw
          * @throws IOException if an i/o-error occurs
          */
-        public void error(final IOException pException) throws IOException { throw pException; }
+        public void error(final IOException pException) throws IOException {
+            throw pException;
+        }
     }
 
     private final List<Observer> observers = new ArrayList<>();