Yeah, now I remember; but I didn't pay attention then!  :-)

Anyway, the print methods handle null String and Object, so we are covered.
This is true for outputting expressions in text mode; evaluating attributes
with expressions is, however, another matter.

BTW, I also think it acceptable to raise NPE.  Let's leave print's in
until someone else objects!  :-)

-Kin-man

> Date: Thu, 11 Sep 2003 08:53:36 +0200
> From: Remy Maucherat <[EMAIL PROTECTED]>
> Subject: Re: cvs commit: 
jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/compiler 
Generator.java
> To: Tomcat Developers List <[EMAIL PROTECTED]>
> 
> [EMAIL PROTECTED] wrote:
> > kinman      2003/09/10 16:11:56
> > 
> >   Modified:    jasper2/src/share/org/apache/jasper/compiler Generator.java
> >   Log:
> >   - Use out.print(expr) instead of out.write(String.valueOf(expr)) for 
outputting
> >     expressions in template texts.
> 
> The problem was when the expression was null, if I remember well the 
> original justification for String.valueOf (of course, it is IMO 
> acceptable to raise a NPE).
> 
> Remy
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to