[Libreoffice-commits] .: 2 commits - basic/source dbaccess/source

[Stephan Bergmann]

 basic/source/uno/namecont.cxx              |    2 +-
 dbaccess/source/ui/uno/copytablewizard.cxx |    6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 67d022ac0ce5e67565e0589f4cd9eb05a8fd5a3c
Author: Abeer Sethi <abeersethi.91 at gmail.com>
Date:   Thu Apr 12 18:05:34 2012 +0530

    Bug 42982 Patch - copytablewizard.cxx

diff --git a/dbaccess/source/ui/uno/copytablewizard.cxx b/dbaccess/source/ui/uno/copytablewizard.cxx
index 5af49b9..3933353 100644
--- a/dbaccess/source/ui/uno/copytablewizard.cxx
+++ b/dbaccess/source/ui/uno/copytablewizard.cxx
@@ -983,7 +983,7 @@ SharedConnection CopyTableWizard::impl_extractConnection_throw( const Reference<
 {
     OSL_PRECOND( m_xSourceConnection.is(), "CopyTableWizard::impl_createSourceStatement_throw: illegal call!" );
     if ( !m_xSourceConnection.is() )
-        throw RuntimeException( ::rtl::OUString(), *const_cast< CopyTableWizard* >( this ) );
+        throw RuntimeException( "CopyTableWizard::impl_createSourceStatement_throw: illegal call!", *const_cast< CopyTableWizard* >( this ));
 
     ::utl::SharedUNOComponent< XPreparedStatement > xStatement;
     switch ( m_nCommandType )
@@ -1029,7 +1029,7 @@ SharedConnection CopyTableWizard::impl_extractConnection_throw( const Reference<
 
     default:
         // this should not have survived initialization phase
-        throw RuntimeException( ::rtl::OUString(), *const_cast< CopyTableWizard* >( this ) );
+        throw RuntimeException("No case matched, this should not have survived the initialization phase", *const_cast< CopyTableWizard* >( this ));
     }
 
     return xStatement;
@@ -1165,7 +1165,7 @@ void CopyTableWizard::impl_copyRows_throw( const Reference< XResultSet >& _rxSou
 {
     OSL_PRECOND( m_xDestConnection.is(), "CopyTableWizard::impl_copyRows_throw: illegal call!" );
     if ( !m_xDestConnection.is() )
-        throw RuntimeException( ::rtl::OUString(), *this );
+        throw RuntimeException( "m_xDestConnection is set to null, CopyTableWizard::impl_copyRows_throw: illegal call!", *this );
 
     Reference< XDatabaseMetaData > xDestMetaData( m_xDestConnection->getMetaData(), UNO_QUERY_THROW );
 
commit c9afb3f5a7f713d34f70b680c5d4ab3db4044d1c
Author: Abeer Sethi <abeersethi.91 at gmail.com>
Date:   Thu Apr 12 17:47:36 2012 +0530

    This incorporates the suggested changes.

diff --git a/basic/source/uno/namecont.cxx b/basic/source/uno/namecont.cxx
index c6ff1ba..384b19c 100644
--- a/basic/source/uno/namecont.cxx
+++ b/basic/source/uno/namecont.cxx
@@ -298,7 +298,7 @@ void SAL_CALL NameContainer::addContainerListener( const Reference< XContainerLi
     throw (RuntimeException)
 {
     if( !xListener.is() )
-        throw RuntimeException();
+        throw RuntimeException("addContainerListener called with null xListener", static_cast< cppu::OWeakObject * >(this));
     Reference< XInterface > xIface( xListener, UNO_QUERY );
     maContainerListeners.addInterface( xIface );
 }