Hullo List,

Having been too excited last time (7+ months) and committed to master way too early, I'm back to basics as I try (once again) to get back into LO dev. Here's a minor cppcheck-found unused variable patch. Please review at your convenience.

Cheers,

Kevin
diff --git a/sc/source/ui/vba/vbarange.cxx b/sc/source/ui/vba/vbarange.cxx
index 99a2308..a1190bd 100644
--- a/sc/source/ui/vba/vbarange.cxx
+++ b/sc/source/ui/vba/vbarange.cxx
@@ -5530,15 +5530,10 @@ ScVbaRange::AutoFill(  const uno::Reference< excel::XRange >& Destination, const
             }
         }
     }
-    ScDocShell* pDocSh= excel::GetDocShellFromRange( mxRange );
 
     FillCmd eCmd = FILL_AUTO;
     FillDateCmd eDateCmd = FILL_DAY;
 
-#ifdef VBA_OOBUILD_HACK
-    double fEndValue =  MAXDOUBLE;
-#endif
-
     if ( Type.hasValue() )
     {
         sal_Int16 nFillType = excel::XlAutoFillType::xlFillDefault;
@@ -5581,6 +5576,8 @@ ScVbaRange::AutoFill(  const uno::Reference< excel::XRange >& Destination, const
         }
     }
 #ifdef VBA_OOBUILD_HACK
+    double fEndValue =  MAXDOUBLE;
+    ScDocShell* pDocSh = excel::GetDocShellFromRange( mxRange );
     pDocSh->GetDocFunc().FillAuto( aSourceRange, NULL, eDir, eCmd, eDateCmd,
                                    nCount, fStep, fEndValue, sal_True, sal_True );
 #endif
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to