Date: prev next · Thread: first prev next last
2012 Archives by date, by thread · List index


On 09/13/2012 12:40 PM, Stephan Bergmann wrote:
causes a warning now that fValue is uninitialized in the assignment

   fPrevValue = fValue;

(in code below the code shown above).  From bPrevString, bPrevEmpy,
bString, and bEmpty always being true, it looks like fPrevValue and
fValue have become unused anyway, but it looks a little suspicious to me
that this code has degraded now to a glorious way of repeatedly calling
WriteCell to only ever write empty cells.  I know too little about this
area to tell whether this is as intended or got broken.  (And the
respective code in sc/source/filter/xml/XMLExportDDELinks.cxx has
diverted sufficiently by now to prevent direct comparison.)

Anyway, fixed the warning for now by reducing the code even further, keeping the current functionality as-is; <http://cgit.freedesktop.org/libreoffice/binfilter/commit/?id=c825a7be7e5920519b485e7a133e97c1e9d93849> "-Werror,-Wuninitialized."

Stephan

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.