approved

Would it be paranoid to add  out.flush() after the out.print? Don't know if
it's necessary but  wouldn't hurt ? Or set the "autoflush" flag to true as
second arg of the
printstream constructor?

        PrintStream out = new PrintStream(mStream);
        out.print(mResourceDefs);



On Wed, May 19, 2010 at 7:52 PM, Max Carlson <[email protected]> wrote:

> Change 20100519-maxcarlson-d by maxcarl...@friendly on 2010-05-19 16:45:20
> PDT
>    in /Users/maxcarlson/openlaszlo/trunk-clean
>    for http://svn.openlaszlo.org/openlaszlo/trunk
>
> Summary: Fix DHTML master sprites
>
> Bugs Fixed: LPP-9016 - regression: DHTML master sprites not working
>
> Technical Reviewer: hminsky
> QA Reviewer: ptw
>
> Details: DHTMLCompiler - Correct comment
>
> DHTMLWriter - Remove extra spaces in mResourceDefs resource table, now that
> it's not being minified by script compiler.  addResourceDefs() writes the
> resource table directly to the outputstream, to ensure it's written first.
>
> Tests: cd laszlo-explorer; lzc --runtime=dhtml explore-nav.lzx.
>  explore-nav.js LzResourceLibrary entries will be on the first line.
>
> Files:
> M       WEB-INF/lps/server/src/org/openlaszlo/sc/DHTMLCompiler.java
> M       WEB-INF/lps/server/src/org/openlaszlo/compiler/DHTMLWriter.java
>
> Changeset:
> http://svn.openlaszlo.org/openlaszlo/patches/20100519-maxcarlson-d.tar
>



-- 
Henry Minsky
Software Architect
[email protected]

Reply via email to