EXTENSIONS : Remove usage of DBG_CTOR and DBG_DTOR.
Valgrind is capable of detecting such bugs. No need for extra macros. Change-Id: Ied33afa8b32149b6b7b1c0725167349e817d6c89 Reviewed-on: https://gerrit.libreoffice.org/8407 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
This commit is contained in:
committed by
Caolán McNamara
parent
88c3b0dfc2
commit
ff71f4e55c
@@ -51,12 +51,8 @@ namespace pcr
|
||||
|
||||
namespace PropertyLineElement = ::com::sun::star::inspection::PropertyLineElement;
|
||||
|
||||
|
||||
//= OBrowserLine
|
||||
|
||||
DBG_NAME(OBrowserLine)
|
||||
|
||||
|
||||
OBrowserLine::OBrowserLine( const OUString& _rEntryName, Window* pParent )
|
||||
:m_sEntryName( _rEntryName )
|
||||
,m_aFtTitle(pParent)
|
||||
@@ -70,7 +66,6 @@ namespace pcr
|
||||
,m_bIndentTitle( false )
|
||||
,m_bReadOnly( false )
|
||||
{
|
||||
DBG_CTOR(OBrowserLine,NULL);
|
||||
m_aFtTitle.Show();
|
||||
}
|
||||
|
||||
@@ -79,8 +74,6 @@ namespace pcr
|
||||
{
|
||||
implHideBrowseButton( true, false );
|
||||
implHideBrowseButton( false, false );
|
||||
|
||||
DBG_DTOR(OBrowserLine,NULL);
|
||||
}
|
||||
|
||||
|
||||
|
@@ -346,7 +346,6 @@ namespace pcr
|
||||
|
||||
//= OBrowserListBox
|
||||
|
||||
DBG_NAME(OBrowserListBox)
|
||||
|
||||
OBrowserListBox::OBrowserListBox( Window* pParent, WinBits nWinStyle)
|
||||
:Control(pParent, nWinStyle| WB_CLIPCHILDREN)
|
||||
@@ -362,7 +361,6 @@ namespace pcr
|
||||
,m_bUpdate(sal_True)
|
||||
,m_pControlContextImpl( new PropertyControlContext_Impl( *this ) )
|
||||
{
|
||||
DBG_CTOR(OBrowserListBox,NULL);
|
||||
|
||||
ListBox aListBox(this,WB_DROPDOWN);
|
||||
aListBox.SetPosSizePixel(Point(0,0),Size(100,100));
|
||||
@@ -391,7 +389,6 @@ namespace pcr
|
||||
Hide();
|
||||
Clear();
|
||||
|
||||
DBG_DTOR(OBrowserListBox,NULL);
|
||||
}
|
||||
|
||||
|
||||
|
@@ -35,14 +35,11 @@ namespace pcr
|
||||
|
||||
//= class OPropertyBrowserView
|
||||
|
||||
DBG_NAME(OPropertyBrowserView)
|
||||
|
||||
OPropertyBrowserView::OPropertyBrowserView(Window* _pParent, WinBits nBits)
|
||||
:Window(_pParent, nBits | WB_3DLOOK)
|
||||
,m_nActivePage(0)
|
||||
{
|
||||
DBG_CTOR(OPropertyBrowserView,NULL);
|
||||
|
||||
m_pPropBox = new OPropertyEditor( this );
|
||||
m_pPropBox->SetHelpId(HID_FM_PROPDLG_TABCTR);
|
||||
m_pPropBox->setPageActivationHandler(LINK(this, OPropertyBrowserView, OnPageActivation));
|
||||
@@ -71,7 +68,6 @@ namespace pcr
|
||||
m_pPropBox = NULL;
|
||||
}
|
||||
|
||||
DBG_DTOR(OPropertyBrowserView, NULL);
|
||||
}
|
||||
|
||||
|
||||
|
@@ -46,12 +46,10 @@ namespace pcr
|
||||
|
||||
//= ButtonNavigationHandler
|
||||
|
||||
DBG_NAME( ButtonNavigationHandler )
|
||||
|
||||
ButtonNavigationHandler::ButtonNavigationHandler( const Reference< XComponentContext >& _rxContext )
|
||||
:ButtonNavigationHandler_Base( _rxContext )
|
||||
{
|
||||
DBG_CTOR( ButtonNavigationHandler, NULL );
|
||||
|
||||
m_xSlaveHandler = css::form::inspection::FormComponentPropertyHandler::create( m_xContext );
|
||||
}
|
||||
@@ -59,7 +57,6 @@ namespace pcr
|
||||
|
||||
ButtonNavigationHandler::~ButtonNavigationHandler( )
|
||||
{
|
||||
DBG_DTOR( ButtonNavigationHandler, NULL );
|
||||
}
|
||||
|
||||
|
||||
|
@@ -51,13 +51,11 @@ namespace pcr
|
||||
|
||||
//= CellBindingPropertyHandler
|
||||
|
||||
DBG_NAME( CellBindingPropertyHandler )
|
||||
|
||||
CellBindingPropertyHandler::CellBindingPropertyHandler( const Reference< XComponentContext >& _rxContext )
|
||||
:CellBindingPropertyHandler_Base( _rxContext )
|
||||
,m_pCellExchangeConverter( new DefaultEnumRepresentation( *m_pInfoService, ::getCppuType( static_cast< sal_Int16* >( NULL ) ), PROPERTY_ID_CELL_EXCHANGE_TYPE ) )
|
||||
{
|
||||
DBG_CTOR( CellBindingPropertyHandler, NULL );
|
||||
}
|
||||
|
||||
|
||||
@@ -88,7 +86,6 @@ namespace pcr
|
||||
|
||||
CellBindingPropertyHandler::~CellBindingPropertyHandler( )
|
||||
{
|
||||
DBG_DTOR( CellBindingPropertyHandler, NULL );
|
||||
}
|
||||
|
||||
|
||||
|
@@ -49,18 +49,15 @@ namespace pcr
|
||||
|
||||
//= EditPropertyHandler
|
||||
|
||||
DBG_NAME( EditPropertyHandler )
|
||||
|
||||
EditPropertyHandler::EditPropertyHandler( const Reference< XComponentContext >& _rxContext )
|
||||
:EditPropertyHandler_Base( _rxContext )
|
||||
{
|
||||
DBG_CTOR( EditPropertyHandler, NULL );
|
||||
}
|
||||
|
||||
|
||||
EditPropertyHandler::~EditPropertyHandler( )
|
||||
{
|
||||
DBG_DTOR( EditPropertyHandler, NULL );
|
||||
}
|
||||
|
||||
|
||||
|
@@ -56,19 +56,16 @@ namespace pcr
|
||||
|
||||
//= EFormsPropertyHandler
|
||||
|
||||
DBG_NAME( EFormsPropertyHandler )
|
||||
|
||||
EFormsPropertyHandler::EFormsPropertyHandler( const Reference< XComponentContext >& _rxContext )
|
||||
:EFormsPropertyHandler_Base( _rxContext )
|
||||
,m_bSimulatingModelChange( false )
|
||||
{
|
||||
DBG_CTOR( EFormsPropertyHandler, NULL );
|
||||
}
|
||||
|
||||
|
||||
EFormsPropertyHandler::~EFormsPropertyHandler( )
|
||||
{
|
||||
DBG_DTOR( EFormsPropertyHandler, NULL );
|
||||
}
|
||||
|
||||
|
||||
|
@@ -355,18 +355,15 @@ namespace pcr
|
||||
ScriptEventDescriptor impl_getDescriptor_throw( const OUString& _rEventName ) const;
|
||||
};
|
||||
|
||||
DBG_NAME( EventHolder )
|
||||
|
||||
EventHolder::EventHolder()
|
||||
{
|
||||
DBG_CTOR( EventHolder, NULL );
|
||||
}
|
||||
|
||||
EventHolder::~EventHolder()
|
||||
{
|
||||
m_aEventNameAccess.clear();
|
||||
m_aEventIndexAccess.clear();
|
||||
DBG_DTOR( EventHolder, NULL );
|
||||
}
|
||||
|
||||
void EventHolder::addEvent( EventId _nId, const OUString& _rEventName, const ScriptEventDescriptor& _rScriptEvent )
|
||||
@@ -456,7 +453,6 @@ namespace pcr
|
||||
return !m_aEventNameAccess.empty();
|
||||
}
|
||||
|
||||
DBG_NAME( EventHandler )
|
||||
|
||||
EventHandler::EventHandler( const Reference< XComponentContext >& _rxContext )
|
||||
:EventHandler_Base( m_aMutex )
|
||||
@@ -466,12 +462,10 @@ namespace pcr
|
||||
,m_bIsDialogElement( false )
|
||||
,m_nGridColumnType( -1 )
|
||||
{
|
||||
DBG_CTOR( EventHandler, NULL );
|
||||
}
|
||||
|
||||
EventHandler::~EventHandler()
|
||||
{
|
||||
DBG_DTOR( EventHandler, NULL );
|
||||
}
|
||||
|
||||
OUString SAL_CALL EventHandler::getImplementationName( ) throw (RuntimeException, std::exception)
|
||||
|
@@ -135,7 +135,6 @@ namespace pcr
|
||||
|
||||
//= FormComponentPropertyHandler
|
||||
|
||||
DBG_NAME( FormComponentPropertyHandler )
|
||||
#define PROPERTY_ID_ROWSET 1
|
||||
|
||||
FormComponentPropertyHandler::FormComponentPropertyHandler( const Reference< XComponentContext >& _rxContext )
|
||||
@@ -148,14 +147,12 @@ namespace pcr
|
||||
,m_bHaveCommand( false )
|
||||
,m_nClassId( 0 )
|
||||
{
|
||||
DBG_CTOR( FormComponentPropertyHandler, NULL );
|
||||
registerProperty(PROPERTY_ROWSET,PROPERTY_ID_ROWSET,0,&m_xRowSet,::getCppuType(&m_xRowSet));
|
||||
}
|
||||
|
||||
|
||||
FormComponentPropertyHandler::~FormComponentPropertyHandler()
|
||||
{
|
||||
DBG_DTOR( FormComponentPropertyHandler, NULL );
|
||||
}
|
||||
|
||||
IMPLEMENT_FORWARD_XINTERFACE2(FormComponentPropertyHandler,FormComponentPropertyHandler_Base,::comphelper::OPropertyContainer)
|
||||
|
@@ -253,12 +253,10 @@ namespace pcr
|
||||
|
||||
//= FormGeometryHandler - implementation
|
||||
|
||||
DBG_NAME( FormGeometryHandler )
|
||||
|
||||
FormGeometryHandler::FormGeometryHandler( const Reference< XComponentContext >& _rxContext )
|
||||
:FormGeometryHandler_Base( _rxContext )
|
||||
{
|
||||
DBG_CTOR( FormGeometryHandler, NULL );
|
||||
}
|
||||
|
||||
|
||||
@@ -270,7 +268,6 @@ namespace pcr
|
||||
dispose();
|
||||
}
|
||||
|
||||
DBG_DTOR( FormGeometryHandler, NULL );
|
||||
}
|
||||
|
||||
|
||||
|
@@ -578,7 +578,6 @@ namespace pcr
|
||||
|
||||
//= DefaultEnumRepresentation
|
||||
|
||||
DBG_NAME( DefaultEnumRepresentation )
|
||||
|
||||
DefaultEnumRepresentation::DefaultEnumRepresentation( const IPropertyInfoService& _rInfo, const Type& _rType, sal_Int32 _nPropertyId )
|
||||
:m_refCount( 0 )
|
||||
@@ -586,13 +585,11 @@ namespace pcr
|
||||
,m_aType( _rType )
|
||||
,m_nPropertyId( _nPropertyId )
|
||||
{
|
||||
DBG_CTOR( DefaultEnumRepresentation, NULL );
|
||||
}
|
||||
|
||||
|
||||
DefaultEnumRepresentation::~DefaultEnumRepresentation()
|
||||
{
|
||||
DBG_DTOR( DefaultEnumRepresentation, NULL );
|
||||
}
|
||||
|
||||
|
||||
|
@@ -218,7 +218,6 @@ namespace pcr
|
||||
void impl_dispatch_throw( const OUString& _rURL );
|
||||
};
|
||||
|
||||
DBG_NAME( UrlClickHandler )
|
||||
|
||||
UrlClickHandler::UrlClickHandler( const Reference<XComponentContext>& _rContext, const Reference< XHyperlinkControl >& _rxControl )
|
||||
:m_xContext( _rContext )
|
||||
@@ -233,12 +232,10 @@ namespace pcr
|
||||
osl_atomic_decrement( &m_refCount );
|
||||
OSL_ENSURE( m_refCount > 0, "UrlClickHandler::UrlClickHandler: leaking!" );
|
||||
|
||||
DBG_CTOR( UrlClickHandler, NULL );
|
||||
}
|
||||
|
||||
UrlClickHandler::~UrlClickHandler()
|
||||
{
|
||||
DBG_DTOR( UrlClickHandler, NULL );
|
||||
}
|
||||
|
||||
void SAL_CALL UrlClickHandler::actionPerformed( const ActionEvent& rEvent ) throw (RuntimeException, std::exception)
|
||||
@@ -277,7 +274,6 @@ namespace pcr
|
||||
xDispatch->dispatch( aURL, aDispatchArgs );
|
||||
}
|
||||
|
||||
DBG_NAME( GenericPropertyHandler )
|
||||
|
||||
GenericPropertyHandler::GenericPropertyHandler( const Reference< XComponentContext >& _rxContext )
|
||||
:GenericPropertyHandler_Base( m_aMutex )
|
||||
@@ -285,14 +281,11 @@ namespace pcr
|
||||
,m_aPropertyListeners( m_aMutex )
|
||||
,m_bPropertyMapInitialized( false )
|
||||
{
|
||||
DBG_CTOR( GenericPropertyHandler, NULL );
|
||||
|
||||
m_xTypeConverter = Converter::create(_rxContext);
|
||||
}
|
||||
|
||||
GenericPropertyHandler::~GenericPropertyHandler()
|
||||
{
|
||||
DBG_DTOR( GenericPropertyHandler, NULL );
|
||||
}
|
||||
|
||||
OUString SAL_CALL GenericPropertyHandler::getImplementationName( ) throw (RuntimeException, std::exception)
|
||||
|
@@ -81,7 +81,6 @@ namespace pcr
|
||||
|
||||
//= OPropertyBrowserController
|
||||
|
||||
DBG_NAME(OPropertyBrowserController)
|
||||
|
||||
OPropertyBrowserController::OPropertyBrowserController( const Reference< XComponentContext >& _rxContext )
|
||||
:m_xContext(_rxContext)
|
||||
@@ -93,7 +92,6 @@ namespace pcr
|
||||
,m_bConstructed( false )
|
||||
,m_bBindingIntrospectee( false )
|
||||
{
|
||||
DBG_CTOR(OPropertyBrowserController,NULL);
|
||||
}
|
||||
|
||||
|
||||
@@ -102,7 +100,6 @@ namespace pcr
|
||||
// stop listening for property changes
|
||||
acquire();
|
||||
stopInspection( true );
|
||||
DBG_DTOR(OPropertyBrowserController,NULL);
|
||||
}
|
||||
|
||||
|
||||
|
@@ -74,7 +74,6 @@ namespace pcr
|
||||
const char* CheckPropertyBrowserInvariants( const void* pVoid );
|
||||
// for dignostics with DBG_CHKTHIS
|
||||
#endif
|
||||
DBG_NAMEEX( OPropertyBrowserController )
|
||||
|
||||
|
||||
//= OPropertyBrowserController
|
||||
|
@@ -40,7 +40,6 @@ namespace pcr
|
||||
|
||||
// class OPropertyEditor
|
||||
|
||||
DBG_NAME(OPropertyEditor)
|
||||
|
||||
OPropertyEditor::OPropertyEditor( Window* pParent, WinBits nWinStyle)
|
||||
:Control(pParent, nWinStyle)
|
||||
@@ -50,7 +49,6 @@ namespace pcr
|
||||
,m_nMinHelpLines( 0 )
|
||||
,m_nMaxHelpLines( 0 )
|
||||
{
|
||||
DBG_CTOR(OPropertyEditor,NULL);
|
||||
|
||||
m_aTabControl.Show();
|
||||
m_aTabControl.SetDeactivatePageHdl(LINK(this, OPropertyEditor, OnPageDeactivate));
|
||||
@@ -64,7 +62,6 @@ namespace pcr
|
||||
{
|
||||
Hide();
|
||||
ClearAll();
|
||||
DBG_DTOR(OPropertyEditor,NULL);
|
||||
}
|
||||
|
||||
|
||||
|
@@ -50,7 +50,6 @@ namespace pcr
|
||||
using namespace ::com::sun::star::inspection;
|
||||
using namespace ::comphelper;
|
||||
|
||||
DBG_NAME( PropertyHandler )
|
||||
|
||||
PropertyHandler::PropertyHandler( const Reference< XComponentContext >& _rxContext )
|
||||
:PropertyHandler_Base( m_aMutex )
|
||||
@@ -59,14 +58,12 @@ namespace pcr
|
||||
,m_xContext( _rxContext )
|
||||
,m_pInfoService ( new OPropertyInfoService )
|
||||
{
|
||||
DBG_CTOR( PropertyHandler, NULL );
|
||||
|
||||
m_xTypeConverter = Converter::create(_rxContext);
|
||||
}
|
||||
|
||||
PropertyHandler::~PropertyHandler()
|
||||
{
|
||||
DBG_DTOR( PropertyHandler, NULL );
|
||||
}
|
||||
|
||||
void SAL_CALL PropertyHandler::inspect( const Reference< XInterface >& _rxIntrospectee ) throw (RuntimeException, NullPointerException, std::exception)
|
||||
|
@@ -46,7 +46,6 @@ namespace pcr
|
||||
|
||||
// OSelectLabelDialog
|
||||
|
||||
DBG_NAME(OSelectLabelDialog)
|
||||
|
||||
OSelectLabelDialog::OSelectLabelDialog( Window* pParent, Reference< XPropertySet > _xControlModel )
|
||||
:ModalDialog(pParent, PcrRes(RID_DLG_SELECTLABELCONTROL))
|
||||
@@ -62,8 +61,6 @@ namespace pcr
|
||||
,m_pLastSelected(NULL)
|
||||
,m_bHaveAssignableControl(sal_False)
|
||||
{
|
||||
DBG_CTOR(OSelectLabelDialog,NULL);
|
||||
|
||||
// initialize the TreeListBox
|
||||
m_aControlTree.SetSelectionMode( SINGLE_SELECTION );
|
||||
m_aControlTree.SetDragDropMode( 0 );
|
||||
@@ -163,7 +160,6 @@ namespace pcr
|
||||
pLoop = m_aControlTree.Next(pLoop);
|
||||
}
|
||||
|
||||
DBG_DTOR(OSelectLabelDialog,NULL);
|
||||
}
|
||||
|
||||
|
||||
|
@@ -86,21 +86,18 @@ namespace pcr
|
||||
|
||||
//= SubmissionPropertyHandler
|
||||
|
||||
DBG_NAME( SubmissionPropertyHandler )
|
||||
|
||||
SubmissionPropertyHandler::SubmissionPropertyHandler( const Reference< XComponentContext >& _rxContext )
|
||||
:EditPropertyHandler_Base( _rxContext )
|
||||
,OPropertyChangeListener( m_aMutex )
|
||||
,m_pPropChangeMultiplexer( NULL )
|
||||
{
|
||||
DBG_CTOR( SubmissionPropertyHandler, NULL );
|
||||
}
|
||||
|
||||
|
||||
SubmissionPropertyHandler::~SubmissionPropertyHandler( )
|
||||
{
|
||||
disposeAdapter();
|
||||
DBG_DTOR( SubmissionPropertyHandler, NULL );
|
||||
}
|
||||
|
||||
|
||||
|
@@ -70,7 +70,6 @@ namespace pcr
|
||||
|
||||
//= TabOrderDialog
|
||||
|
||||
DBG_NAME(TabOrderDialog)
|
||||
|
||||
TabOrderDialog::TabOrderDialog( Window* _pParent, const Reference< XTabControllerModel >& _rxTabModel,
|
||||
const Reference< XControlContainer >& _rxControlCont, const Reference< XComponentContext >& _rxORB )
|
||||
@@ -86,7 +85,6 @@ namespace pcr
|
||||
get(m_pPB_MoveDown, "downB");
|
||||
get(m_pPB_AutoOrder, "autoB");
|
||||
|
||||
DBG_CTOR(TabOrderDialog,NULL);
|
||||
|
||||
m_pPB_MoveUp->SetClickHdl( LINK( this, TabOrderDialog, MoveUpClickHdl ) );
|
||||
m_pPB_MoveDown->SetClickHdl( LINK( this, TabOrderDialog, MoveDownClickHdl ) );
|
||||
@@ -124,7 +122,6 @@ namespace pcr
|
||||
// delete pLB_Controls;
|
||||
delete pImageList;
|
||||
|
||||
DBG_DTOR(TabOrderDialog,NULL);
|
||||
}
|
||||
|
||||
|
||||
@@ -291,12 +288,10 @@ namespace pcr
|
||||
|
||||
//= TabOrderListBox
|
||||
|
||||
DBG_NAME(TabOrderListBox);
|
||||
|
||||
TabOrderListBox::TabOrderListBox( Window* pParent, WinBits nBits )
|
||||
:SvTreeListBox( pParent, nBits )
|
||||
{
|
||||
DBG_CTOR(TabOrderListBox,NULL);
|
||||
SetDragDropMode(0xFFFF/*SV_DRAGDROP_CTRL_MOVE*/);
|
||||
// Hmm. The flag alone is not enough, so to be on the safe side ...
|
||||
|
||||
@@ -316,7 +311,6 @@ namespace pcr
|
||||
|
||||
TabOrderListBox::~TabOrderListBox()
|
||||
{
|
||||
DBG_DTOR(TabOrderListBox,NULL);
|
||||
}
|
||||
|
||||
|
||||
|
@@ -168,13 +168,10 @@ namespace pcr
|
||||
|
||||
// class OFormattedNumericControl
|
||||
|
||||
DBG_NAME(OFormattedNumericControl);
|
||||
|
||||
OFormattedNumericControl::OFormattedNumericControl( Window* pParent, WinBits nWinStyle )
|
||||
:OFormattedNumericControl_Base( PropertyControlType::Unknown, pParent, nWinStyle )
|
||||
{
|
||||
DBG_CTOR(OFormattedNumericControl,NULL);
|
||||
|
||||
getTypedControlWindow()->TreatAsNumber(sal_True);
|
||||
|
||||
m_nLastDecimalDigits = getTypedControlWindow()->GetDecimalDigits();
|
||||
@@ -183,7 +180,6 @@ namespace pcr
|
||||
|
||||
OFormattedNumericControl::~OFormattedNumericControl()
|
||||
{
|
||||
DBG_DTOR(OFormattedNumericControl,NULL);
|
||||
}
|
||||
|
||||
|
||||
|
@@ -71,18 +71,14 @@ namespace pcr
|
||||
|
||||
//= XSDValidationPropertyHandler
|
||||
|
||||
DBG_NAME( XSDValidationPropertyHandler )
|
||||
|
||||
XSDValidationPropertyHandler::XSDValidationPropertyHandler( const Reference< XComponentContext >& _rxContext )
|
||||
:XSDValidationPropertyHandler_Base( _rxContext )
|
||||
{
|
||||
DBG_CTOR( XSDValidationPropertyHandler, NULL );
|
||||
}
|
||||
|
||||
|
||||
XSDValidationPropertyHandler::~XSDValidationPropertyHandler()
|
||||
{
|
||||
DBG_DTOR( XSDValidationPropertyHandler, NULL );
|
||||
}
|
||||
|
||||
|
||||
|
Reference in New Issue
Block a user