Re: [JBoss-dev] Branch_3_2 and jdk 1.3

2004-10-16 Thread Adrian Brock
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); //

Re: [JBoss-dev] Branch_3_2 and jdk 1.3

2004-10-16 Thread Adrian Brock
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); //

[JBoss-dev] Branch_3_2 and jdk 1.3

2004-10-15 Thread Adrian Brock
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.