dbaccess/source/ui/dlg/ConnectionHelper.cxx        |    3 +--
 dbaccess/source/ui/dlg/queryfilter.cxx             |    5 ++---
 dbaccess/source/ui/dlg/tablespage.cxx              |    6 ++----
 dbaccess/source/ui/misc/WCPage.cxx                 |    3 +--
 dbaccess/source/ui/querydesign/JoinTableView.cxx   |    3 +--
 dbaccess/source/ui/querydesign/querycontroller.cxx |    2 +-
 dbaccess/source/ui/tabledesign/TEditControl.cxx    |    3 +--
 dbaccess/source/ui/tabledesign/TableController.cxx |    3 +--
 8 files changed, 10 insertions(+), 18 deletions(-)

New commits:
commit 141469eb831c054d6f4b56fcaa50a03e52429f11
Author: Michael Weghorn <m.wegh...@posteo.de>
Date:   Mon Dec 8 21:09:53 2014 +0100

    fdo#39440 reduce scope of local variables
    
    This addresses some cppcheck warnings.
    
    Change-Id: I8931ff46394923d672e08b3a5c9ab75bb6ffb0a9
    Reviewed-on: https://gerrit.libreoffice.org/13385
    Reviewed-by: Markus Mohrhard <markus.mohrh...@googlemail.com>
    Tested-by: Markus Mohrhard <markus.mohrh...@googlemail.com>

diff --git a/dbaccess/source/ui/dlg/ConnectionHelper.cxx 
b/dbaccess/source/ui/dlg/ConnectionHelper.cxx
index 33863aa..7649be1 100644
--- a/dbaccess/source/ui/dlg/ConnectionHelper.cxx
+++ b/dbaccess/source/ui/dlg/ConnectionHelper.cxx
@@ -534,7 +534,6 @@ namespace dbaui
     IS_PATH_EXIST OConnectionHelper::pathExists(const OUString& _rURL, bool 
bIsFile) const
     {
         ::ucbhelper::Content aCheckExistence;
-        bool bExists = false;
         IS_PATH_EXIST eExists = PATH_NOT_EXIST;
         Reference< ::com::sun::star::task::XInteractionHandler > 
xInteractionHandler(
             task::InteractionHandler::createWithParent(m_xORB, 0), UNO_QUERY );
@@ -545,7 +544,7 @@ namespace dbaui
         try
         {
             aCheckExistence = ::ucbhelper::Content(_rURL, xCmdEnv, 
comphelper::getProcessComponentContext());
-            bExists = bIsFile? aCheckExistence.isDocument(): 
aCheckExistence.isFolder();
+            const bool bExists = bIsFile? aCheckExistence.isDocument(): 
aCheckExistence.isFolder();
             eExists = bExists? PATH_EXIST: PATH_NOT_EXIST;
         }
         catch(const Exception&)
diff --git a/dbaccess/source/ui/dlg/queryfilter.cxx 
b/dbaccess/source/ui/dlg/queryfilter.cxx
index 7c3ad25..1274a03 100644
--- a/dbaccess/source/ui/dlg/queryfilter.cxx
+++ b/dbaccess/source/ui/dlg/queryfilter.cxx
@@ -297,11 +297,11 @@ bool DlgFilterCrit::getCondition(const ListBox& 
_rField,const ListBox& _rComp,co
     try
     {
         OUString sTableName;
-        bool bFunction = false;
         _rFilter.Name = _rField.GetSelectEntry();
         Reference< XPropertySet > xColumn = getQueryColumn(_rFilter.Name);
         if ( xColumn.is() )
         {
+            bool bFunction = false;
             Reference< XPropertySetInfo > xInfo = 
xColumn->getPropertySetInfo();
             if ( xInfo->hasPropertyByName(PROPERTY_REALNAME) )
             {
@@ -818,10 +818,9 @@ void DlgFilterCrit::fillLines(const 
Sequence<Sequence<PropertyValue > >& _aValue
 {
     const Sequence<PropertyValue >* pOrIter = _aValues.getConstArray();
     const Sequence<PropertyValue >* pOrEnd   = pOrIter + _aValues.getLength();
-    bool bOr = true;
     for(sal_uInt16 i=0;pOrIter != pOrEnd; ++pOrIter)
     {
-        bOr = true;
+        bool bOr = true;
         const PropertyValue* pAndIter   = pOrIter->getConstArray();
         const PropertyValue* pAndEnd    = pAndIter + pOrIter->getLength();
         for(;pAndIter != pAndEnd; ++pAndIter)
diff --git a/dbaccess/source/ui/dlg/tablespage.cxx 
b/dbaccess/source/ui/dlg/tablespage.cxx
index 9b8104c..ce9da40 100644
--- a/dbaccess/source/ui/dlg/tablespage.cxx
+++ b/dbaccess/source/ui/dlg/tablespage.cxx
@@ -162,8 +162,6 @@ namespace dbaui
         OUString sCatalog, sSchema, sName;
 
         SvTreeListEntry* pRootEntry = m_pTablesList->getAllObjectsEntry();
-        bool bAllTables = false;
-        bool bAllSchemas = false;
 
         const OUString* pIncludeTable = _rTables.getConstArray();
         for (sal_Int32 i=0; i<_rTables.getLength(); ++i, ++pIncludeTable)
@@ -173,8 +171,8 @@ namespace dbaui
             else
                 sName = pIncludeTable->getStr();
 
-            bAllTables = (1 == sName.getLength()) && ('%' == sName[0]);
-            bAllSchemas = (1 == sSchema.getLength()) && ('%' == sSchema[0]);
+            bool bAllTables = (1 == sName.getLength()) && ('%' == sName[0]);
+            bool bAllSchemas = (1 == sSchema.getLength()) && ('%' == 
sSchema[0]);
 
             // the catalog entry
             SvTreeListEntry* pCatalog = 
m_pTablesList->GetEntryPosByName(sCatalog, pRootEntry);
diff --git a/dbaccess/source/ui/misc/WCPage.cxx 
b/dbaccess/source/ui/misc/WCPage.cxx
index ae647cf..5d2b824 100644
--- a/dbaccess/source/ui/misc/WCPage.cxx
+++ b/dbaccess/source/ui/misc/WCPage.cxx
@@ -277,11 +277,10 @@ bool OCopyTable::checkAppendData()
         ODatabaseExport::TColumnVector::const_iterator aDestIter    = 
rDestColumns.begin();
         ODatabaseExport::TColumnVector::const_iterator aDestEnd     = 
rDestColumns.end();
         const sal_uInt32 nDestSize = rDestColumns.size();
-        bool bNotConvert;
         sal_uInt32 i = 0;
         for(sal_Int32 nPos = 1;aDestIter != aDestEnd && i < nDestSize && i < 
nSrcSize;++aDestIter,++nPos,++i)
         {
-            bNotConvert = true;
+            bool bNotConvert = true;
             m_pParent->m_vColumnPos[i] = 
ODatabaseExport::TPositions::value_type(nPos,nPos);
             TOTypeInfoSP pTypeInfo = 
m_pParent->convertType((*aDestIter)->second->getSpecialTypeInfo(),bNotConvert);
             if ( !bNotConvert )
diff --git a/dbaccess/source/ui/querydesign/JoinTableView.cxx 
b/dbaccess/source/ui/querydesign/JoinTableView.cxx
index 8f6dc07..f622e3e 100644
--- a/dbaccess/source/ui/querydesign/JoinTableView.cxx
+++ b/dbaccess/source/ui/querydesign/JoinTableView.cxx
@@ -525,12 +525,11 @@ void OJoinTableView::SetDefaultTabWinPosSize( 
OTableWindow* pTabWin )
         aRowRect.Bottom() = (nRow+1) * ( TABWIN_SPACING_Y + TABWIN_HEIGHT_STD 
);
 
         // check occupied areas of this line
-        OTableWindow* pOtherTabWin;
         OTableWindowMap::iterator aIter = m_aTableMap.begin();
         OTableWindowMap::iterator aEnd = m_aTableMap.end();
         for(;aIter != aEnd;++aIter)
         {
-            pOtherTabWin = aIter->second;
+            OTableWindow* pOtherTabWin = aIter->second;
             Rectangle aOtherTabWinRect( pOtherTabWin->GetPosPixel(), 
pOtherTabWin->GetSizePixel() );
 
             if(
diff --git a/dbaccess/source/ui/querydesign/querycontroller.cxx 
b/dbaccess/source/ui/querydesign/querycontroller.cxx
index 1b66601da..f3b0251 100644
--- a/dbaccess/source/ui/querydesign/querycontroller.cxx
+++ b/dbaccess/source/ui/querydesign/querycontroller.cxx
@@ -1835,9 +1835,9 @@ void OQueryController::impl_reset( const bool 
i_bForceCurrentControllerSettings
                     xProp->getPropertyValue( PROPERTY_COMMAND ) >>= 
sNewStatement;
                     setStatement_fireEvent( sNewStatement );
 
-                    bool bNewEscapeProcessing( true );
                     if ( editingQuery() )
                     {
+                        bool bNewEscapeProcessing( true );
                         xProp->getPropertyValue( PROPERTY_ESCAPE_PROCESSING ) 
>>= bNewEscapeProcessing;
                         setEscapeProcessing_fireEvent( bNewEscapeProcessing );
                     }
diff --git a/dbaccess/source/ui/tabledesign/TEditControl.cxx 
b/dbaccess/source/ui/tabledesign/TEditControl.cxx
index a9af7d9..ce169f3 100644
--- a/dbaccess/source/ui/tabledesign/TEditControl.cxx
+++ b/dbaccess/source/ui/tabledesign/TEditControl.cxx
@@ -488,11 +488,10 @@ sal_Int32 OTableEditorCtrl::HasFieldName( const OUString& 
rFieldName )
 
     ::std::vector< ::boost::shared_ptr<OTableRow> >::iterator aIter = 
m_pRowList->begin();
     ::std::vector< ::boost::shared_ptr<OTableRow> >::iterator aEnd = 
m_pRowList->end();
-    OFieldDescription* pFieldDescr;
     sal_Int32 nCount(0);
     for(;aIter != aEnd;++aIter)
     {
-        pFieldDescr = (*aIter)->GetActFieldDescr();
+        OFieldDescription* pFieldDescr = (*aIter)->GetActFieldDescr();
         if( pFieldDescr && bCase(rFieldName,pFieldDescr->GetName()))
             nCount++;
     }
diff --git a/dbaccess/source/ui/tabledesign/TableController.cxx 
b/dbaccess/source/ui/tabledesign/TableController.cxx
index ef36c4d..a8acd04 100644
--- a/dbaccess/source/ui/tabledesign/TableController.cxx
+++ b/dbaccess/source/ui/tabledesign/TableController.cxx
@@ -798,7 +798,6 @@ void OTableController::loadData()
         Reference<XColumnsSupplier> xColSup(m_xTable,UNO_QUERY);
         OSL_ENSURE(xColSup.is(),"No XColumnsSupplier!");
         Reference<XNameAccess> xColumns = xColSup->getColumns();
-        OFieldDescription* pActFieldDescr = NULL;
         // ReadOnly-Flag
         // For Drop no row may be editable
         // For Add only the empty rows may be editable
@@ -855,7 +854,7 @@ void OTableController::loadData()
                 pTypeInfo = m_pTypeInfo;
             pTabEdRow->SetFieldType( pTypeInfo, bForce );
 
-            pActFieldDescr = pTabEdRow->GetActFieldDescr();
+            OFieldDescription* pActFieldDescr = pTabEdRow->GetActFieldDescr();
             OSL_ENSURE(pActFieldDescr, "OTableController::loadData: invalid 
field description generated by the table row!");
             if ( pActFieldDescr )
             {
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to