[jira] Created: (LANG-602) ContextedRuntimeException no longer an 'unchecked' exception

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[jira] Created: (LANG-602) ContextedRuntimeException no longer an 'unchecked' exception

Gary D. Gregory (Jira)
ContextedRuntimeException no longer an 'unchecked' exception
------------------------------------------------------------

                 Key: LANG-602
                 URL: https://issues.apache.org/jira/browse/LANG-602
             Project: Commons Lang
          Issue Type: Bug
          Components: lang.exception.*
    Affects Versions: 3.0
            Reporter: Derek C. Ashmore


The purpose of ContextedRuntimeException was to provide an 'unchecked' exception with contexted features.  the fact that it now extends 'java.langException' makes it a checked exception identical to ContextedException and doesn't fulfill its purpose.

Recommend you either just remove ContextedRuntimeException (as it's now redundant with ContextedException) or revert the change that has it extending java.lang.Exception back so that it extends java.lang.RuntimeException.  I would prefer keeping ContextedRuntimeException and having it extend RuntimeException.

I apologize for not catching this earlier.  I only caught it now in attempt to use ContextedRuntimeException on one of my projects and discovering that it is no longer unchecked.

Thanks for looking at this.

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

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (LANG-602) ContextedRuntimeException no longer an 'unchecked' exception

Gary D. Gregory (Jira)

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

Derek C. Ashmore updated LANG-602:
----------------------------------

    Attachment: ContextedRuntimeException.java

If it helps, I've attached a slightly modified ContextedRuntimeException class that fixes the issue -- just changes the class to extend RuntimeException instead of Exception.

Thanks, again, for looking at this.

> ContextedRuntimeException no longer an 'unchecked' exception
> ------------------------------------------------------------
>
>                 Key: LANG-602
>                 URL: https://issues.apache.org/jira/browse/LANG-602
>             Project: Commons Lang
>          Issue Type: Bug
>          Components: lang.exception.*
>    Affects Versions: 3.0
>            Reporter: Derek C. Ashmore
>         Attachments: ContextedRuntimeException.java
>
>
> The purpose of ContextedRuntimeException was to provide an 'unchecked' exception with contexted features.  the fact that it now extends 'java.langException' makes it a checked exception identical to ContextedException and doesn't fulfill its purpose.
> Recommend you either just remove ContextedRuntimeException (as it's now redundant with ContextedException) or revert the change that has it extending java.lang.Exception back so that it extends java.lang.RuntimeException.  I would prefer keeping ContextedRuntimeException and having it extend RuntimeException.
> I apologize for not catching this earlier.  I only caught it now in attempt to use ContextedRuntimeException on one of my projects and discovering that it is no longer unchecked.
> Thanks for looking at this.

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

Reply | Threaded
Open this post in threaded view
|

[jira] Resolved: (LANG-602) ContextedRuntimeException no longer an 'unchecked' exception

Gary D. Gregory (Jira)
In reply to this post by Gary D. Gregory (Jira)

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

Joerg Schaible resolved LANG-602.
---------------------------------

       Resolution: Fixed
    Fix Version/s: 3.0
         Assignee: Joerg Schaible

Thanks for heads-up.

{noformat}
commit -m "ContextedRuntimeException must extend RuntimeException (LANG-602)."
    Sending        src/Commons/proper/lang/src/main/java/org/apache/commons/lang3/exception/ContextedRuntimeException.java
    Transmitting file data ...
    Committed revision 920544.
{noformat}

> ContextedRuntimeException no longer an 'unchecked' exception
> ------------------------------------------------------------
>
>                 Key: LANG-602
>                 URL: https://issues.apache.org/jira/browse/LANG-602
>             Project: Commons Lang
>          Issue Type: Bug
>          Components: lang.exception.*
>    Affects Versions: 3.0
>            Reporter: Derek C. Ashmore
>            Assignee: Joerg Schaible
>             Fix For: 3.0
>
>         Attachments: ContextedRuntimeException.java
>
>
> The purpose of ContextedRuntimeException was to provide an 'unchecked' exception with contexted features.  the fact that it now extends 'java.langException' makes it a checked exception identical to ContextedException and doesn't fulfill its purpose.
> Recommend you either just remove ContextedRuntimeException (as it's now redundant with ContextedException) or revert the change that has it extending java.lang.Exception back so that it extends java.lang.RuntimeException.  I would prefer keeping ContextedRuntimeException and having it extend RuntimeException.
> I apologize for not catching this earlier.  I only caught it now in attempt to use ContextedRuntimeException on one of my projects and discovering that it is no longer unchecked.
> Thanks for looking at this.

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