Let's add a comment into each empty catch block to describe why the
exception is catched and not handled so another developer can see the
reason.

Kiran Ayyagari wrote:
> hi Felix,
> 
>> In LdapCoreSessionConnection.java we have multiple constructs like (see
>> below). The methods declares tho throw 2 Exceptions, but with this
>> construct they are never thrown. Is this on purpose?
> yeah, cause in this implementation of LdapConnection interface there
> is no network layer
> involved and the throws clause is just there to adhere to the
> interface signatures
> 
> can fill the catch block with a log statement if required to avoid
> false positives in the reports
> 
> Kiran Ayyagari

Reply via email to