Mailing-List: contact tomcat-dev-help@jakarta.apache.org; run by ezmlm
Precedence: bulk
Reply-To: tomcat-dev@jakarta.apache.org
Message-ID: <048c01c0f47f$cf958340$1e3a0e0a@kcs.com.au>
From: "Geoff Soutter" <geoff@whitewolf.com.au>
To: <tomcat-dev@jakarta.apache.org>
Subject: [repost] [T4] servlet including jsp causes jsp to flush buffer?
Date: Thu, 14 Jun 2001 13:12:14 +1000
MIME-Version: 1.0
Content-Type: text/plain;
	charset="iso-8859-1"
Content-Transfer-Encoding: 7bit

come on guys, I know you're listening ... :-)

is this really a bug or am I missing something obvious? (I'm not really up
to speed on Jasper...)

If it is I can enter it into bugzilla...

geoff

----- Original Message -----
From: "Geoff Soutter" <geoff@whitewolf.com.au>
To: <tomcat-dev@jakarta.apache.org>
Sent: Wednesday, June 13, 2001 4:14 PM
Subject: [T4] servlet including jsp causes jsp to flush buffer?


> Hi,
>
> I've got a servlet which includes a JSP page, and it appears that the JSP
> engine in Tomcat 4.0 calls response.flushBuffer() when the include
finishes.
>
> IMHO this behaviour is incorrect, as it should be up to the "topmost"
> servlet to control what is happening with the buffering as much as
possible.
> Currently this behaviour is preventing my servlet from controlling the
> buffering as it would like to.
>
> I looked though the list and noticed that v3.x has many flushing problems
> but I was kinda hoping they'd be fixed in Tomcat 4.x ;-).
>
> Anyway, here's the offending code (I think).
>
> In PageContextImpl.java
>
> public void release() {
>   try {
>     if (isIncluded) {
>       ((JspWriterImpl)out).flushBuffer(); // push it into the including
> jspWriter
>     } else {
>       out.flush();
>     }
>   } catch (IOException ex) {
>     loghelper.log("Internal error flushing the buffer in release()");
>   }
>
> and in JspWriterImpl.java
>
> public void flush()  throws IOException {
>   synchronized (lock) {
>     flushBuffer();
>     if (out != null) {
>       out.flush();
>       // Also flush the response buffer.
>       response.flushBuffer();
>     }
>   }
> }
>
> Geoff
>
> --
> Keep cool till after school!
>
>