[jira] [Commented] (NET-153) Add getCause method to CopyStreamException

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (NET-153) Add getCause method to CopyStreamException

Gilles (Jira)

    [ https://issues.apache.org/jira/browse/NET-153?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13093823#comment-13093823 ]

Sebb commented on NET-153:
--------------------------

Yes, the update does seem to have been lost.

However, now that we require Java 1.5, and getCause()/initCause() were added to Throwable in 1.4, would it not be better to use the underlying cause field from Throwable?

i.e. rather than overriding getCause, we should store the cause using initCause(), and update the method as follows:

{code}

public IOException getIOException()
{
    return getCause();
}
{code}

This could further be simplified to merge the initCause() in the super(message) method invocation once Java 1.6 is a minimum requirement.


> Add getCause method to CopyStreamException
> ------------------------------------------
>
>                 Key: NET-153
>                 URL: https://issues.apache.org/jira/browse/NET-153
>             Project: Commons Net
>          Issue Type: Improvement
>    Affects Versions: 1.4
>            Reporter: Dan Godfrey
>            Priority: Trivial
>             Fix For: 2.0
>
>         Attachments: CopyStreamException.patch
>
>
> Add a getCause method to CopyStreamException that has the same signature as Throwable#getCause from JDK 1.4 and returns the wrapped IOException.
> This will override the existing getCause method in version of Java > 1.4 and hence include the IOExceptions stack trace in the CopyStreamExceptions stack trace or just be ignored in Java 1.3.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira