[Libreoffice] [PATCHSET] [PUSHED, partial] calc - removal of dead code

Kohei Yoshida kyoshida at novell.com
Tue Jan 18 10:51:15 PST 2011

On Mon, 2011-01-17 at 20:07 +0100, Thomas Arnhold wrote:
> Here are some more patches for calc. Have fun!

Thanks Thomas.

I've reviewed & applied them, though there were parts that I didn't

>From the 0001 patch, the first several hunks simply failed to apply, so
I only applied from chart2/source/controller/dialogs/DialogModel.cxx
and onward.

I skipped the 0005 patch since it simply didn't apply against the master
branch as of my last update.

>From the 0007 patch, the first few hunks failed, so I only applied
chart2/source/controller/chartapiwrapper/WrappedStockProperties.cxx and

Also, I skipped the following hunk since I was pretty sure that it was
unintentional and the build stopped at that.

@@ -68,7 +68,7 @@
 #include <com/sun/star/util/XModifyBroadcaster.hpp>
 #include <com/sun/star/frame/LayoutManagerEvents.hpp>
 // header for define RET_OK
 #include <vcl/msgbox.hxx>

Other than that, the rest have been applied & pushed.

Thanks a lot!


Kohei Yoshida, LibreOffice hacker, Calc
<kyoshida at novell.com>

More information about the LibreOffice mailing list