https://issues.apache.org/bugzilla/show_bug.cgi?id=46209


Sebb <[EMAIL PROTECTED]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|WONTFIX                     |




--- Comment #8 from Sebb <[EMAIL PROTECTED]>  2008-11-14 13:54:20 PST ---
(In reply to comment #7)
> There is nothing concrete here, just theory and style preferences. not actual
> bugs. If you wish to engage in coding style discussions, bugzilla is not the
> right forum, it would be more proper to take that discussion to the dev list
> 

There *are* still some concrete items which remain to be fixed.

* M X OBL: Method org.apache.tomcat.jdbc.pool.PooledConnection.validate(int,
String) may fail to clean up stream or resource of type java.sql.Statement

==

* M B It: org.apache.tomcat.jdbc.pool.FairBlockingQueue$FairIterator.next()
can't throw NoSuchElementException

However it can throw ArrayOutOfBounds, which should be detected - and probably
converted to NoSuchElement.

==

* M D REC: Exception is caught when Exception is not thrown in
org.apache.tomcat.jdbc.pool.ConnectionPool.getConnection()

Not strictly a bug, but it makes debugging almost impossible if all Exceptions
are converted into SQLException.

==

The issues of thread-safety are not a matter of theory, they are a real
consequence of the Java Memory Model. However explaining that is probably best
left to the list.

==

I will take the other matters to the list.


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to