Re: svn commit: r495895 - /incubator/adffaces/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/io/XhtmlResponseWriter.java

[Adam Winer <aw...@gmail.com>]

Whoa there!   We can't make this change in JSF 1.1.

In JSF 1.1, flush() has a very specific mixed meaning because
of the interweaving of JSP execution and UIComponent rendering.
flush() means "some ordinary JSP output may be coming,
so flush all buffered ResponseWriter content".

In JSF 1.2, flush() can mean something normal.  So,
I think this change needs to be rolled back from 1.1
and applied to 1.2 only (to all ResponseWriters, not
just XhtmlResponseWriter).

-- Adam


On 1/13/07, matzew@apache.org <ma...@apache.org> wrote:
> Author: matzew
> Date: Sat Jan 13 04:54:54 2007
> New Revision: 495895
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=495895
> Log:
> added _out.flush() => ADFFACES-352
>
> Modified:
>     incubator/adffaces/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/io/XhtmlResponseWriter.java
>
> Modified: incubator/adffaces/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/io/XhtmlResponseWriter.java
> URL: http://svn.apache.org/viewvc/incubator/adffaces/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/io/XhtmlResponseWriter.java?view=diff&rev=495895&r1=495894&r2=495895
> ==============================================================================
> --- incubator/adffaces/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/io/XhtmlResponseWriter.java (original)
> +++ incubator/adffaces/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/io/XhtmlResponseWriter.java Sat Jan 13 04:54:54 2007
> @@ -87,6 +87,7 @@
>    public void flush() throws IOException
>    {
>      _closeStartIfNecessary();
> +    _out.flush();
>    }
>
>
>
>
>