[jboss-cvs] JBoss Messaging SVN: r7384 - trunk/tests/src/org/jboss/messaging/tests/unit/core/journal/impl.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Wed Jun 17 18:14:30 EDT 2009


Author: clebert.suconic at jboss.com
Date: 2009-06-17 18:14:29 -0400 (Wed, 17 Jun 2009)
New Revision: 7384

Modified:
   trunk/tests/src/org/jboss/messaging/tests/unit/core/journal/impl/CleanBufferTest.java
Log:
little tweak: fixing a small leak as I realized while looking into something else

Modified: trunk/tests/src/org/jboss/messaging/tests/unit/core/journal/impl/CleanBufferTest.java
===================================================================
--- trunk/tests/src/org/jboss/messaging/tests/unit/core/journal/impl/CleanBufferTest.java	2009-06-17 20:00:17 UTC (rev 7383)
+++ trunk/tests/src/org/jboss/messaging/tests/unit/core/journal/impl/CleanBufferTest.java	2009-06-17 22:14:29 UTC (rev 7384)
@@ -76,35 +76,43 @@
    private void testBuffer(final SequentialFileFactory factory)
    {
       ByteBuffer buffer = factory.newBuffer(100);
-      for (byte b = 0; b < 100; b++)
+      
+      try
       {
-         buffer.put(b);
-      }
-
-      buffer.rewind();
-
-      for (byte b = 0; b < 100; b++)
-      {
-         assertEquals(b, buffer.get());
-      }
-
-      buffer.limit(10);
-      factory.clearBuffer(buffer);
-      buffer.limit(100);
-
-      buffer.rewind();
-
-      for (byte b = 0; b < 100; b++)
-      {
-         if (b < 10)
+         for (byte b = 0; b < 100; b++)
          {
-            assertEquals(0, buffer.get());
+            buffer.put(b);
          }
-         else
+   
+         buffer.rewind();
+   
+         for (byte b = 0; b < 100; b++)
          {
             assertEquals(b, buffer.get());
          }
+   
+         buffer.limit(10);
+         factory.clearBuffer(buffer);
+         buffer.limit(100);
+   
+         buffer.rewind();
+   
+         for (byte b = 0; b < 100; b++)
+         {
+            if (b < 10)
+            {
+               assertEquals(0, buffer.get());
+            }
+            else
+            {
+               assertEquals(b, buffer.get());
+            }
+         }
       }
+      finally
+      {
+         factory.releaseBuffer(buffer);
+      }
    }
 
    // Package protected ---------------------------------------------




More information about the jboss-cvs-commits mailing list