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-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org