[
https://issues.apache.org/jira/browse/SANDBOX-298?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Christian Grobmeier updated SANDBOX-298:
----------------------------------------
Attachment: patch-runcrcerror.txt
Patch attached.
I actually removed the throwCRC method - who really needs a private method
which only throws one IOException? The error is thrown where it actually
happens now.
> BZip2CompressorInputStream.reportCRCError() does not report problems to user
> -----------------------------------------------------------------------------
>
> Key: SANDBOX-298
> URL: https://issues.apache.org/jira/browse/SANDBOX-298
> Project: Commons Sandbox
> Issue Type: Bug
> Components: Compress
> Reporter: Sebb
> Attachments: patch-runcrcerror.txt
>
>
> BZip2CompressorInputStream.reportCRCError() merely writes an error message to
> System.err; it should really throw an IOException as indicated in the method
> signature.
> The IOException message should probably show the expected and actual CRCs.
> Or at least the method should set a flag that can be tested by the caller.
> The code says it does this because "previous versions of the class did" -
> however this does not apply, as there are no previous versions released.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
