loplugin:constantparam in chart2

Change-Id: I9adcf4516107aaa960463be13090c2ac15685e09
Reviewed-on: https://gerrit.libreoffice.org/23833
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
This commit is contained in:
Noel Grandin 2016-04-05 08:24:32 +02:00 committed by Noel Grandin
parent 9866efe3e5
commit a7ec6c29aa
20 changed files with 31 additions and 47 deletions

View File

@ -283,7 +283,7 @@ bool AccessibleBase::ImplUpdateChildren()
}
else if ( aIt->isAdditionalShape() )
{
AddChild( new AccessibleChartShape( aAccInfo, true, false ) );
AddChild( new AccessibleChartShape( aAccInfo ) );
}
}
bResult = true;

View File

@ -50,9 +50,8 @@ namespace chart
AccessibleChartElement::AccessibleChartElement(
const AccessibleElementInfo & rAccInfo,
bool bMayHaveChildren,
bool bAlwaysTransparent /* default: false */ ) :
impl::AccessibleChartElement_Base( rAccInfo, bMayHaveChildren, bAlwaysTransparent ),
bool bMayHaveChildren ) :
impl::AccessibleChartElement_Base( rAccInfo, bMayHaveChildren, false/*bAlwaysTransparent*/ ),
m_bHasText( false )
{
AddState( AccessibleStateType::TRANSIENT );

View File

@ -69,8 +69,7 @@ class AccessibleChartElement :
{
public:
AccessibleChartElement( const AccessibleElementInfo & rAccInfo,
bool bMayHaveChildren,
bool bAlwaysTransparent = false );
bool bMayHaveChildren );
virtual ~AccessibleChartElement();
// ________ AccessibleBase ________

View File

@ -36,9 +36,8 @@ namespace chart
{
AccessibleChartShape::AccessibleChartShape(
const AccessibleElementInfo& rAccInfo,
bool bMayHaveChildren, bool bAlwaysTransparent )
:impl::AccessibleChartShape_Base( rAccInfo, bMayHaveChildren, bAlwaysTransparent )
const AccessibleElementInfo& rAccInfo )
:impl::AccessibleChartShape_Base( rAccInfo, true/*bMayHaveChildren*/, false/*bAlwaysTransparent*/ )
{
if ( rAccInfo.m_aOID.isAdditionalShape() )
{

View File

@ -44,8 +44,7 @@ class AccessibleChartShape :
public impl::AccessibleChartShape_Base
{
public:
AccessibleChartShape( const AccessibleElementInfo& rAccInfo,
bool bMayHaveChildren, bool bAlwaysTransparent = false );
AccessibleChartShape( const AccessibleElementInfo& rAccInfo );
virtual ~AccessibleChartShape();
// ________ XServiceInfo ________

View File

@ -33,7 +33,7 @@ AccessibleBase* ChartElementFactory::CreateChartElement( const AccessibleElement
{
case OBJECTTYPE_DATA_POINT:
case OBJECTTYPE_LEGEND_ENTRY:
return new AccessibleChartElement( rAccInfo, false, false );
return new AccessibleChartElement( rAccInfo, false );
case OBJECTTYPE_PAGE:
case OBJECTTYPE_TITLE:
case OBJECTTYPE_LEGEND:
@ -56,7 +56,7 @@ AccessibleBase* ChartElementFactory::CreateChartElement( const AccessibleElement
case OBJECTTYPE_DATA_STOCK_LOSS:
case OBJECTTYPE_DATA_STOCK_GAIN:
case OBJECTTYPE_DATA_CURVE_EQUATION:
return new AccessibleChartElement( rAccInfo, true, false );
return new AccessibleChartElement( rAccInfo, true );
case OBJECTTYPE_UNKNOWN:
break;
default:

View File

@ -96,7 +96,7 @@ OutputDevice * lcl_GetParentRefDevice( const uno::Reference< frame::XModel > & x
}
DrawViewWrapper::DrawViewWrapper( SdrModel* pSdrModel, OutputDevice* pOut, bool bPaintPageForEditMode)
DrawViewWrapper::DrawViewWrapper( SdrModel* pSdrModel, OutputDevice* pOut)
: E3dView(pSdrModel, pOut)
, m_pMarkHandleProvider(nullptr)
, m_apOutliner(SdrMakeOutliner(OUTLINERMODE_TEXTOBJECT, *pSdrModel))
@ -104,8 +104,7 @@ DrawViewWrapper::DrawViewWrapper( SdrModel* pSdrModel, OutputDevice* pOut, bool
{
SetBufferedOutputAllowed(true);
SetBufferedOverlayAllowed(true);
SetPagePaintingAllowed(bPaintPageForEditMode);
SetPagePaintingAllowed(true);
// #i12587# support for shapes in chart
SdrOutliner* pOutliner = getOutliner();

View File

@ -45,7 +45,7 @@ protected:
class DrawViewWrapper : public E3dView
{
public:
DrawViewWrapper(SdrModel* pModel, OutputDevice* pOut, bool bPaintPageForEditMode);
DrawViewWrapper(SdrModel* pModel, OutputDevice* pOut);
virtual ~DrawViewWrapper();
//triggers the use of an updated first page

View File

@ -785,7 +785,7 @@ void ChartController::impl_createDrawViewController()
{
if( m_pDrawModelWrapper )
{
m_pDrawViewWrapper = new DrawViewWrapper(&m_pDrawModelWrapper->getSdrModel(),m_pChartWindow,true);
m_pDrawViewWrapper = new DrawViewWrapper(&m_pDrawModelWrapper->getSdrModel(),m_pChartWindow);
m_pDrawViewWrapper->attachParentReferenceDevice( getModel() );
}
}

View File

@ -86,8 +86,7 @@ protected:
public:
OOO_DLLPUBLIC_CHARTTOOLS CloseableLifeTimeManager( css::util::XCloseable* pCloseable
, css::lang::XComponent* pComponent
, bool bLongLastingCallsCancelable = false );
, css::lang::XComponent* pComponent );
OOO_DLLPUBLIC_CHARTTOOLS virtual ~CloseableLifeTimeManager();
OOO_DLLPUBLIC_CHARTTOOLS bool impl_isDisposedOrClosed( bool bAssert=true );

View File

@ -29,8 +29,7 @@ class CartesianCoordinateSystem : public BaseCoordinateSystem
public:
explicit CartesianCoordinateSystem(
const css::uno::Reference< css::uno::XComponentContext > & xContext,
sal_Int32 nDimensionCount = 2,
bool bSwapXAndYAxis = false );
sal_Int32 nDimensionCount = 2 );
explicit CartesianCoordinateSystem( const CartesianCoordinateSystem & rSource );
virtual ~CartesianCoordinateSystem();

View File

@ -29,8 +29,7 @@ class PolarCoordinateSystem : public BaseCoordinateSystem
public:
explicit PolarCoordinateSystem(
const css::uno::Reference< css::uno::XComponentContext > & xContext,
sal_Int32 nDimensionCount = 2,
bool bSwapXAndYAxis = false );
sal_Int32 nDimensionCount = 2 );
explicit PolarCoordinateSystem( const PolarCoordinateSystem & rSource );
virtual ~PolarCoordinateSystem();

View File

@ -47,9 +47,8 @@ namespace chart
// explicit
CartesianCoordinateSystem::CartesianCoordinateSystem(
const uno::Reference< uno::XComponentContext > & xContext,
sal_Int32 nDimensionCount /* = 2 */,
bool bSwapXAndYAxis /* = sal_False */ ) :
BaseCoordinateSystem( xContext, nDimensionCount, bSwapXAndYAxis )
sal_Int32 nDimensionCount /* = 2 */ ) :
BaseCoordinateSystem( xContext, nDimensionCount, false/*bSwapXAndYAxis*/ )
{}
CartesianCoordinateSystem::CartesianCoordinateSystem(
@ -115,7 +114,7 @@ css::uno::Sequence< OUString > SAL_CALL CartesianCoordinateSystem::getSupportedS
CartesianCoordinateSystem2d::CartesianCoordinateSystem2d(
const uno::Reference< uno::XComponentContext > & xContext ) :
CartesianCoordinateSystem( xContext, 2, false )
CartesianCoordinateSystem( xContext, 2 )
{}
CartesianCoordinateSystem2d::~CartesianCoordinateSystem2d()
@ -158,7 +157,7 @@ css::uno::Sequence< OUString > SAL_CALL CartesianCoordinateSystem2d::getSupporte
CartesianCoordinateSystem3d::CartesianCoordinateSystem3d(
const uno::Reference< uno::XComponentContext > & xContext ) :
CartesianCoordinateSystem( xContext, 3, false )
CartesianCoordinateSystem( xContext, 3 )
{}
CartesianCoordinateSystem3d::~CartesianCoordinateSystem3d()

View File

@ -47,9 +47,8 @@ namespace chart
// explicit
PolarCoordinateSystem::PolarCoordinateSystem(
const uno::Reference< uno::XComponentContext > & xContext,
sal_Int32 nDimensionCount /* = 2 */,
bool bSwapXAndYAxis /* = sal_False */ ) :
BaseCoordinateSystem( xContext, nDimensionCount, bSwapXAndYAxis )
sal_Int32 nDimensionCount /* = 2 */ ) :
BaseCoordinateSystem( xContext, nDimensionCount, false/*bSwapXAndYAxis*/ )
{}
PolarCoordinateSystem::PolarCoordinateSystem(
@ -115,7 +114,7 @@ css::uno::Sequence< OUString > SAL_CALL PolarCoordinateSystem::getSupportedServi
PolarCoordinateSystem2d::PolarCoordinateSystem2d(
const uno::Reference< uno::XComponentContext > & xContext ) :
PolarCoordinateSystem( xContext, 2, false )
PolarCoordinateSystem( xContext, 2 )
{}
PolarCoordinateSystem2d::~PolarCoordinateSystem2d()
@ -158,7 +157,7 @@ css::uno::Sequence< OUString > SAL_CALL PolarCoordinateSystem2d::getSupportedSer
PolarCoordinateSystem3d::PolarCoordinateSystem3d(
const uno::Reference< uno::XComponentContext > & xContext ) :
PolarCoordinateSystem( xContext, 3, false )
PolarCoordinateSystem( xContext, 3 )
{}
PolarCoordinateSystem3d::~PolarCoordinateSystem3d()

View File

@ -125,8 +125,7 @@ Reference< chart2::XCoordinateSystem > SAL_CALL
uno::RuntimeException, std::exception)
{
Reference< chart2::XCoordinateSystem > xResult(
new CartesianCoordinateSystem(
GetComponentContext(), DimensionCount, /* bSwapXAndYAxis */ false ));
new CartesianCoordinateSystem( GetComponentContext(), DimensionCount ));
for( sal_Int32 i=0; i<DimensionCount; ++i )
{

View File

@ -78,8 +78,7 @@ Reference< chart2::XCoordinateSystem > SAL_CALL
uno::RuntimeException, std::exception)
{
Reference< chart2::XCoordinateSystem > xResult(
new CartesianCoordinateSystem(
GetComponentContext(), DimensionCount, /* bSwapXAndYAxis */ false ));
new CartesianCoordinateSystem( GetComponentContext(), DimensionCount ));
for( sal_Int32 i=0; i<DimensionCount; ++i )
{

View File

@ -66,8 +66,7 @@ Reference< XCoordinateSystem > SAL_CALL
static_cast< ::cppu::OWeakObject* >( this ), 0 );
Reference< XCoordinateSystem > xResult(
new PolarCoordinateSystem(
GetComponentContext(), DimensionCount, /* bSwapXAndYAxis */ false ));
new PolarCoordinateSystem( GetComponentContext(), DimensionCount ));
Reference< XAxis > xAxis( xResult->getAxisByDimension( 0, MAIN_AXIS_INDEX ) );
if( xAxis.is() )

View File

@ -164,8 +164,7 @@ Reference< chart2::XCoordinateSystem > SAL_CALL
uno::RuntimeException, std::exception)
{
Reference< chart2::XCoordinateSystem > xResult(
new PolarCoordinateSystem(
GetComponentContext(), DimensionCount, /* bSwapXAndYAxis */ false ));
new PolarCoordinateSystem( GetComponentContext(), DimensionCount ));
for( sal_Int32 i=0; i<DimensionCount; ++i )
{

View File

@ -183,8 +183,7 @@ Reference< chart2::XCoordinateSystem > SAL_CALL
uno::RuntimeException, std::exception)
{
Reference< chart2::XCoordinateSystem > xResult(
new CartesianCoordinateSystem(
GetComponentContext(), DimensionCount, /* bSwapXAndYAxis */ false ));
new CartesianCoordinateSystem( GetComponentContext(), DimensionCount ));
for( sal_Int32 i=0; i<DimensionCount; ++i )
{

View File

@ -162,9 +162,8 @@ bool LifeTimeManager::dispose()
}
CloseableLifeTimeManager::CloseableLifeTimeManager( css::util::XCloseable* pCloseable
, css::lang::XComponent* pComponent
, bool bLongLastingCallsCancelable )
: LifeTimeManager( pComponent, bLongLastingCallsCancelable )
, css::lang::XComponent* pComponent )
: LifeTimeManager( pComponent, false/*bLongLastingCallsCancelable*/ )
, m_pCloseable(pCloseable)
{
impl_init();