[ 
https://issues.apache.org/jira/browse/DBCP-258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12688081#action_12688081
 ] 

Phil Steitz commented on DBCP-258:
----------------------------------

I don't see the improvement.  Close as WONTFIX?

> [PATCH] ManualPoolingDriverExample - more direct construction of driver and 
> minor comment fix
> ---------------------------------------------------------------------------------------------
>
>                 Key: DBCP-258
>                 URL: https://issues.apache.org/jira/browse/DBCP-258
>             Project: Commons Dbcp
>          Issue Type: Improvement
>    Affects Versions: 1.2.2
>            Reporter: Peter Michaux
>            Priority: Minor
>             Fix For: 1.3
>
>         Attachments: ManualPoolingDriverExamplePatch.diff
>
>   Original Estimate: 0.08h
>  Remaining Estimate: 0.08h
>
> This example has a very indirect way of registering the driver and getting it 
> from the DriverManager.
>   Class.forName("org.apache.commons.dbcp.PoolingDriver");
>   PoolingDriver driver = (PoolingDriver) 
> DriverManager.getDriver("jdbc:apache:commons:dbcp:");
> It is much more direct and clearer to do the following
>   PoolingDriver driver = new PoolingDriver();
>   DriverManager.registerDriver(driver);
> -------
> There were some missing backslashes in the comments about running the example

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

Reply via email to