Hi Ceki.
You forgot the double[] in your last commit.
Regards, Joern.
On Thu, Jul 31, 2008 at 10:51 PM, [EMAIL PROTECTED] wrote:
Author: ceki
Date: Thu Jul 31 22:51:04 2008
New Revision: 1086
Modified:
slf4j/trunk/slf4j-api/src/main/java/org/slf4j/helpers/MessageFormatter.java
Hi Jörn,
At least someone is watching over the code. Cool.
I just realized that toString(primitive[]) methods in Arrays class were
introduced in JDK 1.5. SLF4J needs to work on JDK 1.4 if not 1.3. Yikes.
Jörn Huxhorn wrote:
Hi Ceki.
You forgot the double[] in your last commit.
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
Author: ceki
Date: Fri Aug 1 17:55:26 2008
New Revision: 1087
Modified:
slf4j/trunk/slf4j-api/src/main/java/org/slf4j/helpers/BasicMDCAdapter.java
slf4j/trunk/slf4j-api/src/main/java/org/slf4j/helpers/MessageFormatter.java
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
Jörn Huxhorn wrote:
Damn, you are right!
There's still a problem with your deeplyAppendParameter method: It's not
handling recursive arrays yet.
Yes, indeed.
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
Author: ceki
Date: Fri Aug 1 23:52:22 2008
New Revision: 1089
Added:
slf4j/trunk/slf4j-ext/src/main/java/org/slf4j/LoggerX.java
slf4j/trunk/slf4j-ext/src/test/java/org/slf4j/LoggerXTest.java
Log:
started work on bug 86, http://bugzilla.slf4j.org/show_bug.cgi?id=86
Added: