> 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.
Seems that this would depend entirely on the JSP spec. If newlines have
"significance", then Orion is right and the reference implementation is
broken. Unless the spec further stated that newlines between "certain
code segments" would be ignored.
> It shouldn't break the java code by inserting an erroneous println...
> or am I smoking crack?
You may be smoking crack - users are often the last to know.
tim.
> 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
>
