OK, thanks, I understand that (which is how I fixed the code).
However, I believe this isn't the proper behaviour for JSP.
Orion should eat that newline. This is how other JSP engines behave,
including Sun's reference engine.
It shouldn't break the java code by inserting an erroneous println...
or am I smoking crack?
adam
-----Original Message-----
From: Nick Newman [mailto:[EMAIL PROTECTED]]
Sent: Tuesday, February 27, 2001 11:40 AM
To: Orion-Interest
Subject: Re: simple JSP bug with switch
Hi Adam,
I don't think this is a bug, although I agree the messages are
confusing. Set the development="true" flag in the orion-web.xml and you
get the reason explained more fully. Basically what you have written in
the broken version produces code like this:
switch(1){
// write a new line (but can't be reached!)
case 1:
// write newline, "One", newline
break;
// write a new line (but can't be reached!)
default:
// write newline, "Default", newline
break;
// write a new line (but can't be reached!)
}
and the compiler complains (correctly) that some of the copde can't be reached.
Nick Newman
At 08:11 PM 2/26/01 -0500, you wrote:
>Forgive me if this is a known bug, but I didn't see it in the FAQ or
>anywhere on orionsupport.com.
>
>I am using Orion 1.4.7 and I have tried having Orion use Jikes 1.12 as
>well as all three of the 1.30 JDKs
>{Sun,IBM,Blackdown}. This is all running on RedHat 7.0 Linux 2.4.2.
>
>fixed version (but I shouldn't have to do this...):
>
><% switch(1) { case 1: %>
>One
><% break; default: %>
>Default
><% break; } %>
>
>broken version:
>
><% switch(1) { %>
><% case 1: %>
>One
><% break; %>
><% default: %>
>Default
><% break; %>
><% } %>
>
>Gives the following error (this is the Blackdown error, but they're all
>similar)
>
>500 Internal Server Error
>Error parsing JSP page /jsp/test.jsp
>Syntax error in source
>/jsp/test.jsp.java:29: 'case', 'default' or '}' expected (JSP page line 3)
> if(__coreOut == out)
> __coreOut.write(__staticContent, 0, 1);
> ^
>/jsp/test.jsp.java:29: 'case', 'default' or '}' expected (JSP page line 3)
> if(__coreOut == out)
> __coreOut.write(__staticContent, 0, 1);
>
> ^
>/jsp/test.jsp.java:30: 'case', 'default' or '}' expected (JSP page line 3)
> else
> com.evermind.server.http.EvermindJSPWriter.writeBytes(out,
> __staticContent, 0, 1, null);
>
> ^
>/jsp/test.jsp.java:35: 'case', 'default' or '}' expected (JSP page line 4)
> if(__coreOut == out)
> __coreOut.write(__staticContent, 1, 5);
>
> ^
>/jsp/test.jsp.java:36: 'case', 'default' or '}' expected (JSP page line 4)
> else
> com.evermind.server.http.EvermindJSPWriter.writeBytes(out,
> __staticContent, 1, 5, null);
>
> ^
>/jsp/test.jsp.java:38: 'case', 'default' or '}' expected (JSP page line 4)
> break;
> ^
>/jsp/test.jsp.java:41: 'case', 'default' or '}' expected (JSP page line 6)
> if(__coreOut == out)
> __coreOut.write(__staticContent, 6, 1);
>
> ^
>/jsp/test.jsp.java:42: 'case', 'default' or '}' expected (JSP page line 6)
> else
> com.evermind.server.http.EvermindJSPWriter.writeBytes(out,
> __staticContent, 6, 1, null);
>
> ^
>/jsp/test.jsp.java:47: 'case', 'default' or '}' expected (JSP page line 7)
> if(__coreOut == out)
> __coreOut.write(__staticContent, 7, 9);
>
> ^
>/jsp/test.jsp.java:48: 'case', 'default' or '}' expected (JSP page line 7)
> else
> com.evermind.server.http.EvermindJSPWriter.writeBytes(out,
> __staticContent, 7, 9, null);
>
> ^
>/jsp/test.jsp.java:50: 'case', 'default' or '}' expected (JSP page line 7)
> break;
> ^
>/jsp/test.jsp.java:53: 'case', 'default' or '}' expected (JSP page line 9)
> if(__coreOut == out)
> __coreOut.write(__staticContent, 16, 1);
>
> ^
>/jsp/test.jsp.java:54: 'case', 'default' or '}' expected (JSP page line 9)
> else
> com.evermind.server.http.EvermindJSPWriter.writeBytes(out,
> __staticContent, 16, 1, null);
>
> ^
>13 errors
