[Libreoffice-commits] core.git: basic/source
Arnaud Versini
arnaud.versini at gmail.com
Mon Apr 18 07:21:19 UTC 2016
basic/source/sbx/sbxarray.cxx | 27 ++++-----------------------
1 file changed, 4 insertions(+), 23 deletions(-)
New commits:
commit 0fa6c88007f61176ac707cb5d77fd35cf1521123
Author: Arnaud Versini <arnaud.versini at gmail.com>
Date: Sun Apr 17 20:28:00 2016 +0200
BASIC : Simplify SbxArray
Change-Id: I988fcfed2b03039cbc167eaee1d09bffe42cd411
Reviewed-on: https://gerrit.libreoffice.org/24171
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin at gmail.com>
diff --git a/basic/source/sbx/sbxarray.cxx b/basic/source/sbx/sbxarray.cxx
index f78c376..b6b80c2 100644
--- a/basic/source/sbx/sbxarray.cxx
+++ b/basic/source/sbx/sbxarray.cxx
@@ -124,7 +124,7 @@ SbxVariableRef& SbxArray::GetRef32( sal_uInt32 nIdx )
}
while( mpVarEntries->size() <= nIdx )
{
- mpVarEntries->push_back(SbxVarEntry());
+ mpVarEntries->emplace_back();
}
return (*mpVarEntries)[nIdx].mpVar;
}
@@ -141,7 +141,7 @@ SbxVariableRef& SbxArray::GetRef( sal_uInt16 nIdx )
}
while( mpVarEntries->size() <= nIdx )
{
- mpVarEntries->push_back(SbxVarEntry());
+ mpVarEntries->emplace_back();
}
return (*mpVarEntries)[nIdx].mpVar;
}
@@ -198,22 +198,7 @@ void SbxArray::Put32( SbxVariable* pVar, sal_uInt32 nIdx )
void SbxArray::Put( SbxVariable* pVar, sal_uInt16 nIdx )
{
- if( !CanWrite() )
- SetError( ERRCODE_SBX_PROP_READONLY );
- else
- {
- if( pVar )
- if( eType != SbxVARIANT )
- // Convert no objects
- if( eType != SbxOBJECT || pVar->GetClass() != SbxCLASS_OBJECT )
- pVar->Convert( eType );
- SbxVariableRef& rRef = GetRef( nIdx );
- if( static_cast<SbxVariable*>(rRef) != pVar )
- {
- rRef = pVar;
- SetFlag( SbxFlagBits::Modified );
- }
- }
+ Put32(pVar, nIdx);
}
OUString SbxArray::GetAlias( sal_uInt16 nIdx )
@@ -294,11 +279,7 @@ void SbxArray::Remove32( sal_uInt32 nIdx )
void SbxArray::Remove( sal_uInt16 nIdx )
{
- if( nIdx < mpVarEntries->size() )
- {
- mpVarEntries->erase( mpVarEntries->begin() + nIdx );
- SetFlag( SbxFlagBits::Modified );
- }
+ Remove32(nIdx);
}
void SbxArray::Remove( SbxVariable* pVar )
More information about the Libreoffice-commits
mailing list