> From: Igal Sapir [mailto:isa...@apache.org] > Subject: Re: svn commit: r1842849 - in /tomcat/trunk: java/org/apache/tomcat/jni/Library.java > test/org/apache/tomcat/util/net/TesterSupport.java > test/org/apache/tomcat/util/net/openssl/TestOpenSSLConf.java
> How do you feel about aligning the `else` with the `if`? I think that
> it makes it much clearer which block it refers to, e.g.
> if (a == b) {
> ...
> }
> else if (a == c) {
> ...
> }
> else if (a == d) {
> ...
> }
Please don't - besides wasting space, it still invites errors (albeit
usually caught by an IDE or compiler) when someone inserts code or comments
between the closing brace and the else. (Yes, this is one of my pet peeves,
and I'm very glad Tomcat code isn't written this way.)
- Chuck
THIS COMMUNICATION MAY CONTAIN CONFIDENTIAL AND/OR OTHERWISE PROPRIETARY
MATERIAL and is thus for use only by the intended recipient. If you received
this in error, please contact the sender and delete the e-mail and its
attachments from all computers.
smime.p7s
Description: S/MIME cryptographic signature
