On 06/12/2010 20:49, slaur...@apache.org wrote:
> Author: slaurent
> Date: Mon Dec  6 20:49:14 2010
> New Revision: 1042786
> 
> URL: http://svn.apache.org/viewvc?rev=1042786&view=rev


> @@ -63,7 +72,7 @@ public class ThreadLocalLeakPreventionLi
>          try {
>              Lifecycle lifecycle = event.getLifecycle();
>              if (Lifecycle.AFTER_START_EVENT.equals(event.getType())
> -                    && lifecycle instanceof Server) {
> +                && lifecycle instanceof Server) {
>                  // when the server starts, we register ourself as listener 
> for
>                  // all context
>                  // as well as container event listener so that we know when 
> new

That indenting is now misleading (it was fine before) and the operator
is still at the beginning of the line.


> -            log.error("Exception processing event " + event, e);
> +            String msg =
> +                sm.getString(
> +                    "threadLocalLeakPreventionListener.lifecycleEvent.error",
> +                    event);
> +            log.error(msg, e);

More auto formatting "helpfulness" by the look of it. No reason for
String msg = sm.getString(
not to be on one line.

If we can pull some Eclipse settings together to help folks with this
that would be great.

Mark

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

Reply via email to