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

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
apply.

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
onward.

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

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