You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Christian Grobmeier (JIRA)" <ji...@apache.org> on 2009/01/07 07:26:44 UTC

[jira] Updated: (SANDBOX-262) COMPRESS-REDESIGN: ArchiveStreamFactory does not handle exceptions for createArchiveInputStream(InputStream)

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

Christian Grobmeier updated SANDBOX-262:
----------------------------------------

    Attachment: patch-improve-factories.txt

New patch which eliminates reflection usage in the compress and archiver-factories 

> COMPRESS-REDESIGN: ArchiveStreamFactory does not handle exceptions for createArchiveInputStream(InputStream)
> ------------------------------------------------------------------------------------------------------------
>
>                 Key: SANDBOX-262
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-262
>             Project: Commons Sandbox
>          Issue Type: Bug
>          Components: Compress
>         Environment: N/A
>            Reporter: Tim Pinet
>         Attachments: ArchiveStreamFactory.patch, patch-improve-factories.txt
>
>   Original Estimate: 5h
>  Remaining Estimate: 5h
>
> Reflection handles the checked exceptions for Constructor.newInstance() appropriately for:
> public ArchiveInputStream createArchiveInputStream( final String archiverName, final InputStream out ) throws ArchiveException {...}
> However, the other method:
> public ArchiveInputStream createArchiveInputStream( final InputStream input ) throws IOException {...}
> is not implemented to handle checked constructor exceptions since all Exception catches are not handled. It looks as though this method was not completed. This method should handle exceptions similar to its counterpart.

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