[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