tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Felix Schumacher <felix.schumac...@internetallee.de>
Subject Re: svn commit: r1042482 - in /tomcat/trunk: ./ conf/ java/org/apache/catalina/core/ java/org/apache/catalina/loader/ java/org/apache/tomcat/util/threads/ res/confinstall/ webapps/ webapps/docs/ webapps/docs/config/
Date Tue, 07 Dec 2010 15:20:21 GMT
Am Montag, den 06.12.2010, 22:01 +0100 schrieb Sylvain Laurent:
> > 
> >> +    public void lifecycleEvent(LifecycleEvent event) {
> >> +        try {
> >> +            Lifecycle lifecycle = event.getLifecycle();
> >> +            if (Lifecycle.AFTER_START_EVENT.equals(event.getType())
> >> +                    && lifecycle instanceof Server) {
> > With the operator on the new line it is easy to miss what is going on.
> > Generally, Tomcat style is to put the operator on the first line. e.g.
> > if (Lifecycle.AFTER_START_EVENT.equals(event.getType()) &&
> >        lifecycle instanceof Server) {
> > 
> > There are multiple instances of this throughout the commit.
> 
> Still eclipse. But I did not manage to find a setting to change this behavior :-(
It is "Line Wrapping->Expressions->Binary expressions" [x] Wrap before
operator.

hth
 Felix


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message