Done.
Marcus, would you mind testing it one more time?
Thanks,
Vadim
> -----Original Message-----
> From: Carsten Ziegeler [mailto:[EMAIL PROTECTED]]
> Sent: Wednesday, July 04, 2001 8:08
> To: [EMAIL PROTECTED]
> Subject: AW: [C2 patch] Jikes 1.14
>
>
> > Marcus Crafter wrote:
> >
> > Hi Vadim,
> >
> > On Mon, 2 Jul 2001, Vadim Gritsenko wrote:
> >
> > > I patched Jikes compiler, it works fine for me know. Please
> > test and get back
> > > if you need to modify something more.
> >
> > I've just tested the updated code. On my setup I'm receiving an
> > exception at sitemap compilation time (when I don't include
> > the rt.jar
> > and are intentially forcing jikes to report compile errors)
> >
> > ERROR 99424 [cocoon ] (Thread-32): Error compiling sitemap
> > java.lang.StringIndexOutOfBoundsException: String index out of range: 0
> > at java.lang.String.charAt(String.java:507)
> > at
> > org.apache.cocoon.components.language.programming.java.Jikes.parse
> > Stream(Jikes.java:179)
> > at
> > org.apache.cocoon.components.language.programming.java.AbstractJav
> > aCompiler.getErrors(AbstractJavaCompiler.java:114)
> > at
> > org.apache.cocoon.components.language.programming.java.JavaLanguag
> > e.compile(JavaLanguage.java:191)
> > at
> > org.apache.cocoon.components.language.programming.CompiledProgramm
> > ingLanguage.load(CompiledProgrammingLanguage.java:131)
> > at
> > org.apache.cocoon.components.language.generator.ProgramGeneratorIm
> > pl.generateResource(ProgramGeneratorImpl.java:250)
> > at
> > org.apache.cocoon.components.language.generator.ProgramGeneratorIm
> > pl.load(ProgramGeneratorImpl.java:159)
> > at org.apache.cocoon.sitemap.Handler.run(Handler.java:191)
> > at java.lang.Thread.run(Thread.java:484)
> > ERROR 99424 [cocoon ] (Thread-10): Problem with servlet
> >
> > The line of code is:
> >
> > if ((line == null) || (line.charAt(0) != ' ')) break;
> >
> > On my system (x86 Debian GNU/Linux, jdk 1.3.0, jikes 1.14) the input
> > being parsed when the exception is thrown is actually a string of
> > length zero (in my patch I replaced this test with a test for
> > line.length() == 0).
> >
> > Is this different on your system - are we seeing some platform
> > discrepancies here ??
> >
> Why not combining both tests?
>
> if ((line == null) || (line.length() == 0) || (line.charAt(0) != ' '))
> break;
>
> Carsten
>
> > Cheers,
> >
> > Marcus
> >
> >
> > --
> > .....
> > ,,$$$$$$$$$, Marcus Crafter
> > ;$' '$$$$: Computer Systems Engineer
> > $: $$$$: Open Software Associates GmbH
> > $ o_)$$$: 82-84 Mainzer Landstrasse
> > ;$, _/\ &&:' 60327 Frankfurt Germany
> > ' /( &&&
> > \_&&&&' Email : [EMAIL PROTECTED]
> > &&&&. Business Hours : +49 69 9757 200
> > &&&&&&&: After Hours : +49 69 49086750
> >
> >
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, email: [EMAIL PROTECTED]
