Return-Path: Delivered-To: apmail-jakarta-tomcat-dev-archive@apache.org Received: (qmail 62257 invoked from network); 24 Oct 2002 18:07:50 -0000 Received: from unknown (HELO nagoya.betaversion.org) (192.18.49.131) by daedalus.apache.org with SMTP; 24 Oct 2002 18:07:50 -0000 Received: (qmail 21779 invoked by uid 97); 24 Oct 2002 18:08:28 -0000 Delivered-To: qmlist-jakarta-archive-tomcat-dev@jakarta.apache.org Received: (qmail 21714 invoked by uid 97); 24 Oct 2002 18:08:27 -0000 Mailing-List: contact tomcat-dev-help@jakarta.apache.org; run by ezmlm Precedence: bulk List-Unsubscribe: List-Subscribe: List-Help: List-Post: List-Id: "Tomcat Developers List" Reply-To: "Tomcat Developers List" Delivered-To: mailing list tomcat-dev@jakarta.apache.org Received: (qmail 21632 invoked by uid 98); 24 Oct 2002 18:08:26 -0000 X-Antivirus: nagoya (v4218 created Aug 14 2002) To: tomcat-dev@jakarta.apache.org X-Injected-Via-Gmane: http://gmane.org/ Path: not-for-mail From: Costin Manolache Subject: Re: jasper questions Date: Thu, 24 Oct 2002 11:04:23 -0700 Lines: 34 Message-ID: References: <3DB8342A.9060404@apache.org> NNTP-Posting-Host: 64.84.39.162 Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7Bit X-Trace: main.gmane.org 1035482787 24398 64.84.39.162 (24 Oct 2002 18:06:27 GMT) X-Complaints-To: usenet@main.gmane.org NNTP-Posting-Date: Thu, 24 Oct 2002 18:06:27 +0000 (UTC) User-Agent: KNode/0.7.1 Sender: news X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N Remy Maucherat wrote: >> If we let the servlet wrapper deal with it - then we need a 'partial >> flush' >> in release() - which will just move the data to the parent buffer ( >> OutputBuffer from what I see ), but without calling flush on it. >> >> Remy, what do you think ? > > Yes, I think I agree. I rarely see any reason to flush, it just makes > things slower. I think it's a very good idea to see *all* the code from > Jasper 1 with a very critical eye and not assume it's there for a Good > Reason ;-) > > Did you test it ? Does it work good ? (I don't see why it wouldn't, just > wondering) It seems ok - I'm more worried about the spec implications, i.e. is the flush() in relase() implied by some of the spec - or javadoc ? My reading is not, but others understand the spec better. For now I'll just change PageContextImpl.release to do flushBuffer in both cases ( i.e. included or not ). This will empty the page buffers in the OutBuffer. My reading of the javadoc is that release() shouldn't do that ( but discard it )- but that's what has been done in the past and is the least risky. If we agree on the release() behavior, then we need to move the flushBuffer() in the calling code ( the generated servlet ), before release() is called. -- Costin -- To unsubscribe, e-mail: For additional commands, e-mail: