That is interesting.  Seems like a silly thing to do just for a debug
statement.

On Wed, Mar 18, 2009 at 12:05 PM, Devan Goodwin <[email protected]> wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On Wed, 18 Mar 2009 11:57:00 -0400
> Justin Sherrill <[email protected]> wrote:
>
> >
> > Might be a nice thing to do after the next fork for Sat.  Having to
> > use log.isDebugEnabled() is ugly IMO and it'd be nice not to have to
> > do it where we don't have to.
>
> I hate doing this too. I was gonna take issue with it as it's ugly and
> seemed pretty silly until I googled and saw that it is best practice if
> the statement is doing string contact.
>
> Cheers,
>
> Devan
>
> - --
>  Devan Goodwin <[email protected]>
>  Software Engineer     Spacewalk / RHN Satellite
>  Halifax, Canada       650.567.9039x79267
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v2.0.9 (GNU/Linux)
>
> iEYEARECAAYFAknBG7sACgkQAyHWaPV9my7NmwCg4KNXj7yWOtiKunCAywIgYPkX
> yqwAn0fG+PANkMCW/fBj6AwqynQRKayq
> =ebH1
> -----END PGP SIGNATURE-----
>
> _______________________________________________
> Spacewalk-devel mailing list
> [email protected]
> https://www.redhat.com/mailman/listinfo/spacewalk-devel
>
_______________________________________________
Spacewalk-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/spacewalk-devel

Reply via email to