[Libreoffice-commits] .: 3 commits - salhelper/inc stoc/source
Libreoffice Gerrit user
logerrit at kemper.freedesktop.org
Fri Dec 7 02:40:32 PST 2012
salhelper/inc/salhelper/dynload.hxx | 8
stoc/source/defaultregistry/defaultregistry.cxx | 147 ++++++++-------
stoc/source/implementationregistration/implreg.cxx | 3
stoc/source/inspect/introspection.cxx | 4
stoc/source/javavm/jvmargs.cxx | 15 -
stoc/source/registry_tdprovider/rdbtdp_tdenumeration.cxx | 3
6 files changed, 93 insertions(+), 87 deletions(-)
New commits:
commit 14acb6b69800c0850157c2ba15b50c5527e46939
Author: Stephan Bergmann <sbergman at redhat.com>
Date: Fri Dec 7 11:38:05 2012 +0100
Unused variables
Change-Id: I8d0d72ac2c3d7f0b0576980c770cb62af983d074
diff --git a/stoc/source/implementationregistration/implreg.cxx b/stoc/source/implementationregistration/implreg.cxx
index fa3867d..2de4c88 100644
--- a/stoc/source/implementationregistration/implreg.cxx
+++ b/stoc/source/implementationregistration/implreg.cxx
@@ -1657,7 +1657,6 @@ Sequence< OUString > ImplementationRegistration::getImplementations(
const OUString & locationUrl)
throw ( RuntimeException )
{
- OUString implLoaderUrl(implementationLoaderUrl);
OUString activatorName;
if (!implementationLoaderUrl.isEmpty())
@@ -1668,7 +1667,7 @@ Sequence< OUString > ImplementationRegistration::getImplementations(
} else
{
// check locationUrl to find out what kind of loader is needed
- // set implLoaderUrl
+ // set implementationLoaderUrl
}
if( m_xSMgr.is() ) {
diff --git a/stoc/source/inspect/introspection.cxx b/stoc/source/inspect/introspection.cxx
index b8d73d4..5956088 100644
--- a/stoc/source/inspect/introspection.cxx
+++ b/stoc/source/inspect/introspection.cxx
@@ -356,10 +356,6 @@ sal_Int32 IntrospectionAccessStatic_Impl::getMethodIndex( const ::rtl::OUString&
for( int i = 0 ; i < nLen ; ++i )
{
const Reference<XIdlMethod> xMethod2 = pMethods[ i ];
-
- ::rtl::OUString aTestClassName = xMethod2->getDeclaringClass()->getName();
- ::rtl::OUString aTestMethodName = xMethod2->getName();
-
if( xMethod2->getName() == aPureMethodName )
{
Reference< XIdlClass > xMethClass2 = xMethod2->getDeclaringClass();
diff --git a/stoc/source/javavm/jvmargs.cxx b/stoc/source/javavm/jvmargs.cxx
index 31808a6..7497fcb 100644
--- a/stoc/source/javavm/jvmargs.cxx
+++ b/stoc/source/javavm/jvmargs.cxx
@@ -21,8 +21,6 @@
#include "jvmargs.hxx"
#include <rtl/ustring.hxx>
-
-using ::rtl::OUString;
namespace stoc_javavm {
JVM::JVM() throw()//: _enabled(sal_False)
@@ -31,20 +29,9 @@ JVM::JVM() throw()//: _enabled(sal_False)
void JVM::pushProp(const OUString & property)
{
- sal_Int32 index = property.indexOf((sal_Unicode)'=');
- if(index > 0)
- {
- OUString left = property.copy(0, index).trim();
- OUString right(property.copy(index + 1).trim());
- _props.push_back(property);
- }
- else
- { // no '=', could be -X
- _props.push_back(property);
- }
+ _props.push_back(property);
}
-
const ::std::vector< ::rtl::OUString > & JVM::getProperties() const
{
return _props;
diff --git a/stoc/source/registry_tdprovider/rdbtdp_tdenumeration.cxx b/stoc/source/registry_tdprovider/rdbtdp_tdenumeration.cxx
index 20e08ae..50be0cb 100644
--- a/stoc/source/registry_tdprovider/rdbtdp_tdenumeration.cxx
+++ b/stoc/source/registry_tdprovider/rdbtdp_tdenumeration.cxx
@@ -128,9 +128,6 @@ TypeDescriptionEnumerationImpl::createInstance(
aBytes.getConstArray(), aBytes.getLength(), false,
TYPEREG_VERSION_1);
- rtl::OUString aName(
- aReader.getTypeName().replace( '/', '.' ) );
-
if ( aReader.getTypeClass() == RT_TYPE_MODULE )
{
// Do not close xKey!
commit 98108afb23f04d46c81b8b7d97fc2c71f57207a8
Author: Stephan Bergmann <sbergman at redhat.com>
Date: Fri Dec 7 11:31:51 2012 +0100
Proper indentation
Change-Id: I9fe211563c8b2b2718be25180294421af5593886
diff --git a/stoc/source/defaultregistry/defaultregistry.cxx b/stoc/source/defaultregistry/defaultregistry.cxx
index 149593f..b6534fb 100644
--- a/stoc/source/defaultregistry/defaultregistry.cxx
+++ b/stoc/source/defaultregistry/defaultregistry.cxx
@@ -185,8 +185,8 @@ NestedKeyImpl::NestedKeyImpl( NestedRegistryImpl* pDefaultRegistry,
if (m_localKey.is())
{
m_name = m_localKey->getKeyName();
- } else
- if (m_defaultKey.is())
+ }
+ else if (m_defaultKey.is())
{
m_name = m_defaultKey->getKeyName();
}
@@ -213,8 +213,8 @@ NestedKeyImpl::NestedKeyImpl( const OUString& rKeyName,
if (m_localKey.is())
{
m_name = m_localKey->getKeyName();
- } else
- if (m_defaultKey.is())
+ }
+ else if (m_defaultKey.is())
{
m_name = m_defaultKey->getKeyName();
}
@@ -261,10 +261,10 @@ OUString NestedKeyImpl::computeName(const OUString& name)
if ( m_localKey.is() && m_localKey->isValid() )
{
resLocalName = m_localKey->getResolvedName(name);
- } else
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
- return m_defaultKey->getResolvedName(name);
+ return m_defaultKey->getResolvedName(name);
}
if ( !resLocalName.isEmpty() && m_pRegistry->m_defaultReg->isValid() )
@@ -333,8 +333,8 @@ RegistryKeyType SAL_CALL NestedKeyImpl::getKeyType( const OUString& rKeyName )
if ( m_localKey.is() && m_localKey->isValid() )
{
return m_localKey->getKeyType(rKeyName);
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
return m_defaultKey->getKeyType(rKeyName);
}
@@ -352,8 +352,8 @@ RegistryValueType SAL_CALL NestedKeyImpl::getValueType( )
if ( m_localKey.is() && m_localKey->isValid() )
{
return m_localKey->getValueType();
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
return m_defaultKey->getValueType();
}
@@ -371,11 +371,12 @@ sal_Int32 SAL_CALL NestedKeyImpl::getLongValue( )
if ( m_localKey.is() && m_localKey->isValid() )
{
return m_localKey->getLongValue();
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
return m_defaultKey->getLongValue();
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -391,14 +392,15 @@ void SAL_CALL NestedKeyImpl::setLongValue( sal_Int32 value )
if ( m_localKey.is() && m_localKey->isValid() )
{
m_localKey->setLongValue(value);
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
Reference<XRegistryKey> rootKey(m_pRegistry->m_localReg->getRootKey());
m_localKey = rootKey->createKey(m_name);
m_localKey->setLongValue(value);
m_state = m_pRegistry->m_state++;
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -414,11 +416,12 @@ Sequence< sal_Int32 > SAL_CALL NestedKeyImpl::getLongListValue( )
if ( m_localKey.is() && m_localKey->isValid() )
{
return m_localKey->getLongListValue();
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
return m_defaultKey->getLongListValue();
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -434,14 +437,15 @@ void SAL_CALL NestedKeyImpl::setLongListValue( const Sequence< sal_Int32 >& seqV
if ( m_localKey.is() && m_localKey->isValid() )
{
m_localKey->setLongListValue(seqValue);
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
Reference<XRegistryKey> rootKey(m_pRegistry->m_localReg->getRootKey());
m_localKey = rootKey->createKey(m_name);
m_localKey->setLongListValue(seqValue);
m_state = m_pRegistry->m_state++;
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -457,11 +461,12 @@ OUString SAL_CALL NestedKeyImpl::getAsciiValue( )
if ( m_localKey.is() && m_localKey->isValid() )
{
return m_localKey->getAsciiValue();
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
return m_defaultKey->getAsciiValue();
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -477,14 +482,15 @@ void SAL_CALL NestedKeyImpl::setAsciiValue( const OUString& value )
if ( m_localKey.is() && m_localKey->isValid() )
{
m_localKey->setAsciiValue(value);
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
Reference<XRegistryKey> rootKey(m_pRegistry->m_localReg->getRootKey());
m_localKey = rootKey->createKey(m_name);
m_localKey->setAsciiValue(value);
m_state = m_pRegistry->m_state++;
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -500,11 +506,12 @@ Sequence< OUString > SAL_CALL NestedKeyImpl::getAsciiListValue( )
if ( m_localKey.is() && m_localKey->isValid() )
{
return m_localKey->getAsciiListValue();
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
return m_defaultKey->getAsciiListValue();
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -520,14 +527,15 @@ void SAL_CALL NestedKeyImpl::setAsciiListValue( const Sequence< OUString >& seqV
if ( m_localKey.is() && m_localKey->isValid() )
{
m_localKey->setAsciiListValue(seqValue);
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
Reference<XRegistryKey> rootKey(m_pRegistry->m_localReg->getRootKey());
m_localKey = rootKey->createKey(m_name);
m_localKey->setAsciiListValue(seqValue);
m_state = m_pRegistry->m_state++;
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -543,11 +551,12 @@ OUString SAL_CALL NestedKeyImpl::getStringValue( )
if ( m_localKey.is() && m_localKey->isValid() )
{
return m_localKey->getStringValue();
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
return m_defaultKey->getStringValue();
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -563,14 +572,15 @@ void SAL_CALL NestedKeyImpl::setStringValue( const OUString& value )
if ( m_localKey.is() && m_localKey->isValid() )
{
m_localKey->setStringValue(value);
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
Reference<XRegistryKey> rootKey(m_pRegistry->m_localReg->getRootKey());
m_localKey = rootKey->createKey(m_name);
m_localKey->setStringValue(value);
m_state = m_pRegistry->m_state++;
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -586,11 +596,12 @@ Sequence< OUString > SAL_CALL NestedKeyImpl::getStringListValue( )
if ( m_localKey.is() && m_localKey->isValid() )
{
return m_localKey->getStringListValue();
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
return m_defaultKey->getStringListValue();
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -606,14 +617,15 @@ void SAL_CALL NestedKeyImpl::setStringListValue( const Sequence< OUString >& seq
if ( m_localKey.is() && m_localKey->isValid() )
{
m_localKey->setStringListValue(seqValue);
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
Reference<XRegistryKey> rootKey(m_pRegistry->m_localReg->getRootKey());
m_localKey = rootKey->createKey(m_name);
m_localKey->setStringListValue(seqValue);
m_state = m_pRegistry->m_state++;
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -629,11 +641,12 @@ Sequence< sal_Int8 > SAL_CALL NestedKeyImpl::getBinaryValue( )
if ( m_localKey.is() && m_localKey->isValid() )
{
return m_localKey->getBinaryValue();
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
return m_defaultKey->getBinaryValue();
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -649,14 +662,15 @@ void SAL_CALL NestedKeyImpl::setBinaryValue( const Sequence< sal_Int8 >& value )
if ( m_localKey.is() && m_localKey->isValid() )
{
m_localKey->setBinaryValue(value);
- } else
- if ( m_defaultKey.is() && m_defaultKey->isValid() )
+ }
+ else if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
Reference<XRegistryKey> rootKey(m_pRegistry->m_localReg->getRootKey());
m_localKey = rootKey->createKey(m_name);
m_localKey->setBinaryValue(value);
m_state = m_pRegistry->m_state++;
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -691,7 +705,8 @@ Reference< XRegistryKey > SAL_CALL NestedKeyImpl::openKey( const OUString& aKeyN
if ( localKey.is() || defaultKey.is() )
{
return ((XRegistryKey*)new NestedKeyImpl(m_pRegistry, localKey, defaultKey));
- } else
+ }
+ else
{
return Reference<XRegistryKey>();
}
@@ -729,7 +744,8 @@ Reference< XRegistryKey > SAL_CALL NestedKeyImpl::createKey( const OUString& aKe
return ((XRegistryKey*)new NestedKeyImpl(m_pRegistry, localKey, defaultKey));
}
- } else
+ }
+ else
{
Reference<XRegistryKey> localKey, defaultKey;
@@ -785,7 +801,8 @@ void SAL_CALL NestedKeyImpl::deleteKey( const OUString& rKeyName )
}
m_pRegistry->m_localReg->getRootKey()->deleteKey(resolvedName);
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -966,7 +983,8 @@ sal_Bool SAL_CALL NestedKeyImpl::createLink( const OUString& aLinkName, const OU
}
resolvedName = resolvedName + aLinkName.copy(lastIndex);
- } else
+ }
+ else
{
if ( lastIndex == 0 )
resolvedName = m_name + aLinkName;
@@ -977,7 +995,8 @@ sal_Bool SAL_CALL NestedKeyImpl::createLink( const OUString& aLinkName, const OU
if ( m_localKey.is() && m_localKey->isValid() )
{
isCreated = m_pRegistry->m_localReg->getRootKey()->createLink(resolvedName, aLinkTarget);
- } else
+ }
+ else
{
if ( m_defaultKey.is() && m_defaultKey->isValid() )
{
@@ -1020,7 +1039,8 @@ void SAL_CALL NestedKeyImpl::deleteLink( const OUString& rLinkName )
}
resolvedName = resolvedName + rLinkName.copy(lastIndex);
- } else
+ }
+ else
{
if ( lastIndex == 0 )
resolvedName = m_name + rLinkName;
@@ -1032,7 +1052,8 @@ void SAL_CALL NestedKeyImpl::deleteLink( const OUString& rLinkName )
!m_localKey->isReadOnly() )
{
m_pRegistry->m_localReg->getRootKey()->deleteLink(resolvedName);
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
@@ -1064,7 +1085,8 @@ OUString SAL_CALL NestedKeyImpl::getLinkTarget( const OUString& rLinkName )
}
resolvedName = resolvedName + rLinkName.copy(lastIndex);
- } else
+ }
+ else
{
if ( lastIndex == 0 )
resolvedName = m_name + rLinkName;
@@ -1324,7 +1346,8 @@ Reference< XRegistryKey > SAL_CALL NestedRegistryImpl::getRootKey( )
return ((XRegistryKey*)new NestedKeyImpl(this, localKey, defaultKey));
}
- } else
+ }
+ else
{
throw InvalidRegistryException();
}
commit 55a58245c9608cee41c852ac29d432cb611871ab
Author: Stephan Bergmann <sbergman at redhat.com>
Date: Fri Dec 7 11:27:33 2012 +0100
Proper indentation
Change-Id: I4f85be9a33be6d9a3c756f2422eb7574beac1e1d
diff --git a/salhelper/inc/salhelper/dynload.hxx b/salhelper/inc/salhelper/dynload.hxx
index 6def1d7..f79bb22 100644
--- a/salhelper/inc/salhelper/dynload.hxx
+++ b/salhelper/inc/salhelper/dynload.hxx
@@ -156,9 +156,13 @@ public:
if( m_pLoader != toAssign.m_pLoader )
{
if( toAssign.m_pLoader )
- toAssign.m_pLoader->acquire();
+ {
+ toAssign.m_pLoader->acquire();
+ }
if( m_pLoader )
- m_pLoader->release();
+ {
+ m_pLoader->release();
+ }
m_pLoader = toAssign.m_pLoader;
}
More information about the Libreoffice-commits
mailing list