Le 8/02/2016 22:56, ggreg...@apache.org a écrit :
> Modified: 
> commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/DelegatingPreparedStatement.java
> URL: 
> http://svn.apache.org/viewvc/commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/DelegatingPreparedStatement.java?rev=1729274&r1=1729273&r2=1729274&view=diff
> ==============================================================================
> --- 
> commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/DelegatingPreparedStatement.java
>  (original)
> +++ 
> commons/proper/dbcp/trunk/src/main/java/org/apache/commons/dbcp2/DelegatingPreparedStatement.java
>  Mon Feb  8 21:56:48 2016
> @@ -82,7 +82,7 @@ public class DelegatingPreparedStatement
>          try {
>              return 
> DelegatingResultSet.wrapResultSet(this,((PreparedStatement)getDelegate()).executeQuery());
>          }
> -        catch (SQLException e) {
> +        catch (final SQLException e) {
>              handleException(e);
>              throw new AssertionError();
>          }

Am I the only one to think this syntax is uselessly verbose ? I'm ok
with final class variables, less with final local parameters, but final
exceptions look really useless to me.

Emmanuel Bourg


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to