db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yun Lee (JIRA)" <j...@apache.org>
Subject [jira] Updated: (DERBY-4732) Release system resources in CanonTestCase thoroughly
Date Tue, 06 Jul 2010 13:37:49 GMT

     [ https://issues.apache.org/jira/browse/DERBY-4732?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel

Yun Lee updated DERBY-4732:

    Attachment: derby-4732.stat

In Derby-4732.patch, cannonReader and testOutput are closed in finally block to make sure
they can be closed. Besides, rawBytes is nulled out in compareCanon() method and tearDown()
method is removed.

> Release system resources in CanonTestCase thoroughly
> ----------------------------------------------------
>                 Key: DERBY-4732
>                 URL: https://issues.apache.org/jira/browse/DERBY-4732
>             Project: Derby
>          Issue Type: Improvement
>          Components: Test
>            Reporter: Yun Lee
>            Assignee: Yun Lee
>            Priority: Minor
>         Attachments: derby-4732.patch, derby-4732.stat
> Now, in the method of compareCanon(String canon) in CanonTestcase, two BufferedReaders----cannonReader
and testOutput are closed in try catch block, however if some exception occurs above, the
two BufferReaders won't be closed, and related system resources won't be released. 
> Besides, a releative problem is about the ByteArrayOutputStream rawBytes in CanonTestcase,
it's created in getOutputStream() and closed in compareCanon(String canon). It's pointed out
by Knut in Derby-2708, ByteArrayOutputStream's javadoc says that closing it has no effect,
so tearDown() method has been added  to CanonTestCase and sets rawBytes to null. It certainly
can work. However, is it also OK to null out rawBytes in compareCanon(String canon)? If so,
it seems cleaner than to null it out in tearDown() method.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message