----- Original Message -----
From: "Denis Benoit" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Tuesday, April 30, 2002 8:52 PM
Subject: Modification of Jasper2's Generator.java


> I was reading Remy's comments on the Generator.java patch and I noted two
> things:
>
> 1. It could be possible to not always create a Vector and a BitSet if we
>    declare them as instance variables instead of local variables.  This
>    way we could defer their initialisation in the addTagToVector method;
>

No, if they are instance variables, then you quickly fall into thread-safety
hell.  If you have 20 different people accessing the same JSP page at the
same time, then you have no clue as to which tag is in the Vector.

> 2. It was not necessary to call the addTagToVector method in the
>    generateCustomEnd.  In this method we are merely accessing the tag
whose
>    index was popped from the stack.  Since we call addTagToVector when
>    we begin the pseudo "try" block, it follows that the tag is already
>    in the vector when we reach the pseudo "finally" block;
>
> Thanks!
>
> --
> Denis Benoit
> [EMAIL PROTECTED]
>
>
> --
> To unsubscribe, e-mail:
<mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail:
<mailto:[EMAIL PROTECTED]>
>


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

Reply via email to