[slf4j-dev] svn commit: r1086 - in slf4j/trunk/slf4j-api/src: main/java/org/slf4j/helpers test/java/org/slf4j/helpers

[Jörn Huxhorn]

Damn, you are right!
I'm not watching out for 1.5 since I have a >=1.5 requirement for Lilith.

There's still a problem with your deeplyAppendParameter method: It's not
handling recursive arrays yet.

While this isn't a terribly realistic case it would cause a stack overflow
at the moment. Take a look at the 1.5 implementation of deepToString, it's
using a HashSet (called dejaVu :D) to prevent this.

Yes, I'm watching your code... I hope that doesn't make you paranoid ;)

Joern.

On Fri, Aug 1, 2008 at 4:18 PM, Ceki Gulcu <listid at qos.ch> wrote:

>
>
> Jörn Huxhorn wrote:
> > And I just found out that it would be more appropriate to use
> > Arrays.deepToString((Object[]) o); in case of Object[].
>
> Right, except that deepToString is JDK 1.5. Double yikes.
>
> --
> Ceki Gülcü
>
> QOS.ch is looking to hire talented developers located in Switzerland
> to work on cutting-edge software projects. If you think you are
> qualified, then please contact ceki at qos.ch.
> _______________________________________________
> dev mailing list
> dev at slf4j.org
> http://www.slf4j.org/mailman/listinfo/dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://qos.ch/pipermail/slf4j-dev/attachments/20080801/26b6496b/attachment.htm>