Review Request: Fix crashes in the DOC filter

Sebastian Sauer mail at dipe.org
Fri Sep 2 12:50:01 BST 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102520/
-----------------------------------------------------------

Review request for Calligra and Matus Uzak.


Summary
-------

Attached patch fixes the both crash bugs https://bugs.kde.org/show_bug.cgi?id=281112 and https://bugs.kde.org/show_bug.cgi?id=281164 .

I know it is a rather dirty way (see comments I added to bug 281112 for more details) but then
1) it fixes the crashes
2) it doesn't reopen the original problem that code tried to fix
3) it makes it more easy to understand the code (I seriously have no idea what that new code is supposed to do - hint: a comment explaining what it does for what reason would be *veryyyyyyyyyyy* useful here)


Diffs
-----

  filters/words/msword-odf/wv2/src/headers.cpp 877f557 

Diff: http://git.reviewboard.kde.org/r/102520/diff


Testing
-------


Thanks,

Sebastian

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20110902/eddadcd4/attachment.htm>


More information about the calligra-devel mailing list