Merge pull request #1 from apache/trunk

mgmechanics · mgmechanics · commit f36751732219 · 2015-02-24T17:57:42.000+01:00
update from origin
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
@@ -46,6 +46,9 @@ The <action> type attribute can be add,update,fix,remove.
   <body>
 
     <release version="1.0" date="TBA" description="TBA">
+      <action issue="IMAGING-166" dev="britter" type="update" due-to="Michael Groß">
+        ByteSourceImageTest: Local variable hides a field
+      </action>
       <action issue="IMAGING-156" dev="britter" type="update">
         Refactor TagConstants
       </action>
diff --git a/src/test/java/org/apache/commons/imaging/common/bytesource/ByteSourceImageTest.java b/src/test/java/org/apache/commons/imaging/common/bytesource/ByteSourceImageTest.java
@@ -107,13 +107,12 @@ public void test() throws Exception {
     }
 
     public void checkGetBufferedImage(final File file, final byte[] bytes) throws Exception {
-        BufferedImage imageFile = Imaging.getBufferedImage(file);
-        assertNotNull(imageFile);
-        assertTrue(imageFile.getWidth() > 0);
-        assertTrue(imageFile.getHeight() > 0);
-        final int imageFileWidth = imageFile.getWidth();
-        final int imageFileHeight = imageFile.getHeight();
-        imageFile = null;
+        BufferedImage bufferedImage = Imaging.getBufferedImage(file);
+        assertNotNull(bufferedImage);
+        assertTrue(bufferedImage.getWidth() > 0);
+        assertTrue(bufferedImage.getHeight() > 0);
+        final int imageFileWidth = bufferedImage.getWidth();
+        final int imageFileHeight = bufferedImage.getHeight();
 
         final BufferedImage imageBytes = Imaging.getBufferedImage(bytes);
         assertNotNull(imageBytes);
