Peter Royal wrote:
> Just to voice an "egads what a PITA", I use the Cornerstone ConnectionHandler 
> code, and I had a class that extends AbstractConnectionHandlerFactory to 
> create my ConnectionHandlers. My ConnectionHandler used to extend 
> AbstractLoggable. I just noticed a NPE after doing the upgrade a few days ago 
> (just finally tested out that piece) and saw that it was failing on the 
> getLogger() because a logger was never set now that 
> AbstractConnectionHandlerFactory extends LogEnabled and the setupLogger() 
> function only checks for LogEnabled.
> 
> Thus my question/comment: Could AbstractLoggable be marked 'deprecated' and 
> thus situations like this would be evident from compiler warnings?
> -pete


I thought AbstractLoggable and Loggable were marked deprecated.




-- 

"They that give up essential liberty to obtain a little temporary safety
  deserve neither liberty nor safety."
                 - Benjamin Franklin


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

Reply via email to