james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Oleg Kalnichevski (JIRA)" <server-...@james.apache.org>
Subject [jira] Commented: (MIME4J-50) InputBuffer to extend FilterInputStream
Date Mon, 14 Jul 2008 00:07:31 GMT

    [ https://issues.apache.org/jira/browse/MIME4J-50?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12613217#action_12613217

Oleg Kalnichevski commented on MIME4J-50:

Works for me.


> InputBuffer to extend FilterInputStream
> ---------------------------------------
>                 Key: MIME4J-50
>                 URL: https://issues.apache.org/jira/browse/MIME4J-50
>             Project: Mime4j
>          Issue Type: Improvement
>    Affects Versions: 0.3
>            Reporter: Stefano Bagnara
>             Fix For: 0.4
>         Attachments: MIME4J-50.patch
> I create this issue to attach a patch (not sure our mailing list support this).
> I was looking at InputBuffer and I noticed how much it looks like a FilterInputStream
class but it doesn't expose the InputStream "interface" (ok, it is not an interface, but an
abstract class).
> Am I missing something or extending InputStreamFilter does not add complexity to the
class while increasing its flexibility/reusability?
> I gave this a go by making BufferingInputStream an extension of FilterInputStream and
by letting InputBuffer extending the "new" BufferingInputStream.
> This also gave me the opportunity to simplify some code aroud avoiding multiple wrapping
of the InputBuffer.
> After this patch all of the tests still pass.
> I did not rename "InputBuffer" with this patch, but maybe it should include Buffered,
InputStream and Filter words to better autodocumentation.

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

To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org

View raw message