<br><br><div class="gmail_quote">On Wed, Feb 1, 2012 at 3:09 PM,  <span dir="ltr">&lt;<a href="mailto:bugzilla-daemon@freedesktop.org">bugzilla-daemon@freedesktop.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<a href="https://bugs.freedesktop.org/show_bug.cgi?id=39428" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=39428</a><br>
<br>
--- Comment #9 from Caolán McNamara &lt;<a href="mailto:caolanm@redhat.com">caolanm@redhat.com</a>&gt; 2012-02-01 07:09:32 PST ---<br>
here&#39;s one place I know about where the current std::stream-alike &quot;failed reads<br>
leave the original value untouched&quot; is relied upon<br>
<br>
sw/source/filter/ww8/ww8san.cxx<br>
    while( 1 )<br>
    {<br>
        sal_uInt8 clxt(2);<br>
        *pStr &gt;&gt; clxt; //if read fails, then value remains 2, so loop breaks<br>
even on failure<br>
        nLeft--;<br>
        if( 2 == clxt )<br>
            break;<br>
        ...<br>
<div class="HOEnZb"><div class="h5">    }<br>
<br></div></div></blockquote></div>Sorry for the delay in replying. I have changed the return types from Stream&amp; to be the same as the parameter type and a return of 0 indicates failure. Currently working through tools/ <br>
As this is quite a large change, should I submit patches as I complete the various sub-projects?<br><br>Cheers<br><br>