You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@mina.apache.org by Bernd Fondermann <be...@googlemail.com> on 2011/01/10 06:52:01 UTC

[Vysper] Debug outputs

Hi Niklas,

When reviewing the latest changes on the codebase I encountered two
code bits I'm not sure about whether or not they are intended to last:

+ StanzaLoggingFilter: "if (serverLogger.isDebugEnabled()) " is
commented out. Discard, re-enable or leave it that way?
+ ProtocolWorker l. 92: "System.out.println(stanzaHandler);" Left over
from debugging?

  Bernd

Re: [Vysper] Debug outputs

Posted by Niklas Gustavsson <ni...@protocol7.com>.

On Mon, Jan 10, 2011 at 6:52 AM, Bernd Fondermann
<be...@googlemail.com> wrote:
> When reviewing the latest changes on the codebase I encountered two
> code bits I'm not sure about whether or not they are intended to last:
>
> + StanzaLoggingFilter: "if (serverLogger.isDebugEnabled()) " is
> commented out. Discard, re-enable or leave it that way?
> + ProtocolWorker l. 92: "System.out.println(stanzaHandler);" Left over
> from debugging?

Both of these were temporary (and ProtocolWorker not meant for
committing :-). Both should be fixed now. Thanks for reminding me!

/niklas