You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tika.apache.org by "Keith R. Bennett (JIRA)" <ji...@apache.org> on 2007/10/14 19:46:50 UTC

[jira] Created: (TIKA-64) Would like a TikaException(Throwable cause) constructor.

Would like a TikaException(Throwable cause) constructor.
--------------------------------------------------------

                 Key: TIKA-64
                 URL: https://issues.apache.org/jira/browse/TIKA-64
             Project: Tika
          Issue Type: Improvement
          Components: general
    Affects Versions: 0.1-incubator
            Reporter: Keith R. Bennett
            Priority: Trivial
             Fix For: 0.1-incubator


To complete the set of Exception constructors overridden by TikaException, and to simplify the creation of TIkaException in some cases, I suggest adding the following to TikaException.java:

    public TikaException(Throwable cause) {
        super(cause);
    }
 

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


[jira] Commented: (TIKA-64) Would like a TikaException(Throwable cause) constructor.

Posted by "Jukka Zitting (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/TIKA-64?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12534669 ] 

Jukka Zitting commented on TIKA-64:
-----------------------------------

We already have TikaException(String message, Throwable cause).

IMHO an exception should always be accompanied with a message (not just a copy of the underlying exception message), so I'd rather not add the proposed constructor.

> Would like a TikaException(Throwable cause) constructor.
> --------------------------------------------------------
>
>                 Key: TIKA-64
>                 URL: https://issues.apache.org/jira/browse/TIKA-64
>             Project: Tika
>          Issue Type: Improvement
>          Components: general
>    Affects Versions: 0.1-incubator
>            Reporter: Keith R. Bennett
>            Priority: Trivial
>             Fix For: 0.1-incubator
>
>
> To complete the set of Exception constructors overridden by TikaException, and to simplify the creation of TIkaException in some cases, I suggest adding the following to TikaException.java:
>     public TikaException(Throwable cause) {
>         super(cause);
>     }
>  

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


[jira] Closed: (TIKA-64) Would like a TikaException(Throwable cause) constructor.

Posted by "Keith R. Bennett (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TIKA-64?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Keith R. Bennett closed TIKA-64.
--------------------------------

    Resolution: Won't Fix

This was not a bug, but an intentional design decision, to discourage the throwing of a TikaException without a message.

> Would like a TikaException(Throwable cause) constructor.
> --------------------------------------------------------
>
>                 Key: TIKA-64
>                 URL: https://issues.apache.org/jira/browse/TIKA-64
>             Project: Tika
>          Issue Type: Improvement
>          Components: general
>    Affects Versions: 0.1-incubator
>            Reporter: Keith R. Bennett
>            Priority: Trivial
>             Fix For: 0.1-incubator
>
>
> To complete the set of Exception constructors overridden by TikaException, and to simplify the creation of TIkaException in some cases, I suggest adding the following to TikaException.java:
>     public TikaException(Throwable cause) {
>         super(cause);
>     }
>  

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


[jira] Commented: (TIKA-64) Would like a TikaException(Throwable cause) constructor.

Posted by "Bertrand Delacretaz (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/TIKA-64?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12534753 ] 

Bertrand Delacretaz commented on TIKA-64:
-----------------------------------------

I concur with Jukka, I was going to make the same comment. I don't like Exceptions without messages.

> Would like a TikaException(Throwable cause) constructor.
> --------------------------------------------------------
>
>                 Key: TIKA-64
>                 URL: https://issues.apache.org/jira/browse/TIKA-64
>             Project: Tika
>          Issue Type: Improvement
>          Components: general
>    Affects Versions: 0.1-incubator
>            Reporter: Keith R. Bennett
>            Priority: Trivial
>             Fix For: 0.1-incubator
>
>
> To complete the set of Exception constructors overridden by TikaException, and to simplify the creation of TIkaException in some cases, I suggest adding the following to TikaException.java:
>     public TikaException(Throwable cause) {
>         super(cause);
>     }
>  

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