On Fri, 2010-11-26 at 14:12 +0100, Joachim Trémouroux wrote:
> Pierre-André,
> 
> Are you sure it is safe to remove the OpenFlagRec() call ?

Indeed, good catch Joachim. The warning is that the ret value of
OpenFlagRec is unused, looking closed at OpenFlagRec you'll see that it
extracts a value from the underlying stream, so that stream is advanced
by 2 bytes, dropping the call would change that logic. The right fix is
to just drop the BYTE cDFlags =. 

I've fixed this now. Michael has already pushed the remainder of this
patch at least.

C.

_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to