commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Niall Pemberton (JIRA)" <>
Subject [jira] Updated: (DBCP-310) SQLNestedException & use of initCause() with SQLException
Date Sat, 28 Nov 2009 19:59:20 GMT


Niall Pemberton updated DBCP-310:

    Attachment: DBCP-310-v2.patch

OK, then we should consistently use the ,initCause(e) method and remove all references that
throw SQLNestedException.

Attaching a new patch

> SQLNestedException & use of initCause() with SQLException
> ---------------------------------------------------------
>                 Key: DBCP-310
>                 URL:
>             Project: Commons Dbcp
>          Issue Type: Improvement
>            Reporter: Niall Pemberton
>            Priority: Minor
>             Fix For: 1.3, 1.4
>         Attachments: DBCP-310-v2.patch, DBCP-310-v2.patch, SQLNestedException.patch
> SQLNestedException in its current state is a hangover from supporting JDK 1.3 when there
was no "initCause()" method. This implementation can now be greatly simplified with DBCP now
having a minimum of JDK 1.4
> Also SQLNestedException is deprecated and the following code has been used in a number
of places
> {code}
>      throw (SQLException )new SQLException(message).initCause(e)
> {code}
> DBCP is inconsistent though - sometimes using the above and sometimes using SQLNestedException.
IMO SQLNestedException  should be un-deprecated and used consistently everywhere - its now
a simple implementation and I think the code is cleaner using it rather than the above.

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

View raw message