Also we don't
catch (Exception e)
{
e.printStackTrace();
}
and even worse
catch (Exception ignored)
{
// Nothing (eating exceptions is usually bad)
}
It should be
catch (Exception e)
{
log.warn(Some message, e); // for something unexpected or
log.trace(Some message, e); //
Also we don't
catch (Exception e)
{
e.printStackTrace();
}
and even worse
catch (Exception ignored)
{
// Nothing (eating exceptions is usually bad)
}
It should be
catch (Exception e)
{
log.warn(Some message, e); // for something unexpected or
log.trace(Some message, e); //
This is a friendly reminder for those who don't know about this
constraint.
JBoss-3.2.x is a j2ee1.3 implementation and as such, it must be
able to run on j2se 1.3. You cannot use java apis from j2se 1.4.
I'm going to go through the current codebase and remove all
usage of j2se 1.4 apis.