diff options
author | Noel Grandin <noel@peralex.com> | 2015-08-06 16:49:55 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2015-08-07 07:20:24 +0200 |
commit | 6343754e310a589cb49e2a1da0cd68472571179d (patch) | |
tree | a0915b9679f3422a6abda3fb2d6917df4edb30f5 /slideshow/source/engine | |
parent | dea885f80a80c6a5839ee5dbf8521487186a9522 (diff) |
cppcheck:noExplicitConstructor
Change-Id: I2717d3d120df8af5b7e9af8d147a57b9f5b5ecbe
Diffstat (limited to 'slideshow/source/engine')
25 files changed, 35 insertions, 35 deletions
diff --git a/slideshow/source/engine/activities/activitybase.hxx b/slideshow/source/engine/activities/activitybase.hxx index 52ac9663a973..62c183f1ebaa 100644 --- a/slideshow/source/engine/activities/activitybase.hxx +++ b/slideshow/source/engine/activities/activitybase.hxx @@ -37,7 +37,7 @@ namespace internal { class ActivityBase : public AnimationActivity { public: - ActivityBase( const ActivityParameters& rParms ); + explicit ActivityBase( const ActivityParameters& rParms ); /// From Disposable interface virtual void dispose() SAL_OVERRIDE; diff --git a/slideshow/source/engine/activities/continuousactivitybase.hxx b/slideshow/source/engine/activities/continuousactivitybase.hxx index 5c72372d06f9..12a30415241b 100644 --- a/slideshow/source/engine/activities/continuousactivitybase.hxx +++ b/slideshow/source/engine/activities/continuousactivitybase.hxx @@ -36,7 +36,7 @@ namespace slideshow class ContinuousActivityBase : public SimpleContinuousActivityBase { public: - ContinuousActivityBase( const ActivityParameters& rParms ); + explicit ContinuousActivityBase( const ActivityParameters& rParms ); using SimpleContinuousActivityBase::perform; diff --git a/slideshow/source/engine/activities/continuouskeytimeactivitybase.hxx b/slideshow/source/engine/activities/continuouskeytimeactivitybase.hxx index 150e83c8fb9c..10b1ece7f3ec 100644 --- a/slideshow/source/engine/activities/continuouskeytimeactivitybase.hxx +++ b/slideshow/source/engine/activities/continuouskeytimeactivitybase.hxx @@ -38,7 +38,7 @@ namespace slideshow class ContinuousKeyTimeActivityBase : public SimpleContinuousActivityBase { public: - ContinuousKeyTimeActivityBase( const ActivityParameters& rParms ); + explicit ContinuousKeyTimeActivityBase( const ActivityParameters& rParms ); using SimpleContinuousActivityBase::perform; diff --git a/slideshow/source/engine/activities/discreteactivitybase.hxx b/slideshow/source/engine/activities/discreteactivitybase.hxx index 530cb7a48ac6..165e9c394a8d 100644 --- a/slideshow/source/engine/activities/discreteactivitybase.hxx +++ b/slideshow/source/engine/activities/discreteactivitybase.hxx @@ -38,7 +38,7 @@ namespace slideshow class DiscreteActivityBase : public ActivityBase { public: - DiscreteActivityBase( const ActivityParameters& rParms ); + explicit DiscreteActivityBase( const ActivityParameters& rParms ); /** Hook for derived classes. diff --git a/slideshow/source/engine/activities/interpolation.hxx b/slideshow/source/engine/activities/interpolation.hxx index 6effd085e0a3..2418f59c6d10 100644 --- a/slideshow/source/engine/activities/interpolation.hxx +++ b/slideshow/source/engine/activities/interpolation.hxx @@ -88,7 +88,7 @@ namespace slideshow /// Specialization for HSLColor, to employ color-specific interpolator template<> struct Interpolator< HSLColor > { - Interpolator( bool bCCW ) : + explicit Interpolator( bool bCCW ) : mbCCW( bCCW ) { } diff --git a/slideshow/source/engine/activities/simplecontinuousactivitybase.hxx b/slideshow/source/engine/activities/simplecontinuousactivitybase.hxx index a0b968e940ce..473082c04304 100644 --- a/slideshow/source/engine/activities/simplecontinuousactivitybase.hxx +++ b/slideshow/source/engine/activities/simplecontinuousactivitybase.hxx @@ -37,7 +37,7 @@ namespace slideshow class SimpleContinuousActivityBase : public ActivityBase { public: - SimpleContinuousActivityBase( const ActivityParameters& rParms ); + explicit SimpleContinuousActivityBase( const ActivityParameters& rParms ); virtual double calcTimeLag() const SAL_OVERRIDE; virtual bool perform() SAL_OVERRIDE; diff --git a/slideshow/source/engine/animationfactory.cxx b/slideshow/source/engine/animationfactory.cxx index c03c26d312c9..f01dc2d1fc32 100644 --- a/slideshow/source/engine/animationfactory.cxx +++ b/slideshow/source/engine/animationfactory.cxx @@ -619,7 +619,7 @@ namespace slideshow class Scaler { public: - Scaler( double nScale ) : + explicit Scaler( double nScale ) : mnScale( nScale ) { } diff --git a/slideshow/source/engine/effectrewinder.cxx b/slideshow/source/engine/effectrewinder.cxx index af589c2bc0e7..00f51464ee0d 100644 --- a/slideshow/source/engine/effectrewinder.cxx +++ b/slideshow/source/engine/effectrewinder.cxx @@ -45,7 +45,7 @@ class RewinderEventHandler : public EventHandler { public: typedef ::std::function<bool ()> Action; - RewinderEventHandler (const Action& rAction) : maAction(rAction) {} + explicit RewinderEventHandler (const Action& rAction) : maAction(rAction) {} virtual ~RewinderEventHandler() {} private: const Action maAction; @@ -58,7 +58,7 @@ class RewinderAnimationEventHandler : public AnimationEventHandler { public: typedef ::std::function<bool (const AnimationNodeSharedPtr& rpNode)> Action; - RewinderAnimationEventHandler (const Action& rAction) : maAction(rAction) {} + explicit RewinderAnimationEventHandler (const Action& rAction) : maAction(rAction) {} virtual ~RewinderAnimationEventHandler() {} private: const Action maAction; diff --git a/slideshow/source/engine/expressionnodefactory.cxx b/slideshow/source/engine/expressionnodefactory.cxx index 9b6bffe42e88..d4ec3c99285a 100644 --- a/slideshow/source/engine/expressionnodefactory.cxx +++ b/slideshow/source/engine/expressionnodefactory.cxx @@ -42,7 +42,7 @@ namespace slideshow class ConstantValueExpression : public ExpressionNode { public: - ConstantValueExpression( double rValue ) : + explicit ConstantValueExpression( double rValue ) : maValue( rValue ) { } diff --git a/slideshow/source/engine/shapes/drawshape.cxx b/slideshow/source/engine/shapes/drawshape.cxx index 3eec85e8c47c..4d9f64ca16e5 100644 --- a/slideshow/source/engine/shapes/drawshape.cxx +++ b/slideshow/source/engine/shapes/drawshape.cxx @@ -759,7 +759,7 @@ namespace slideshow class Expander { public: - Expander( ::basegfx::B2DSize& rBounds ) : + explicit Expander( ::basegfx::B2DSize& rBounds ) : mrBounds( rBounds ) { } diff --git a/slideshow/source/engine/shapes/drawshapesubsetting.cxx b/slideshow/source/engine/shapes/drawshapesubsetting.cxx index 953f97bdfe74..ad3f7ea37537 100644 --- a/slideshow/source/engine/shapes/drawshapesubsetting.cxx +++ b/slideshow/source/engine/shapes/drawshapesubsetting.cxx @@ -642,7 +642,7 @@ namespace slideshow class CountClassFunctor { public: - CountClassFunctor( DrawShapeSubsetting::IndexClassificator eClass ) : + explicit CountClassFunctor( DrawShapeSubsetting::IndexClassificator eClass ) : meClass( eClass ), mnCurrCount(0) { diff --git a/slideshow/source/engine/slide/layer.cxx b/slideshow/source/engine/slide/layer.cxx index 1e4550bb7642..1d0568343796 100644 --- a/slideshow/source/engine/slide/layer.cxx +++ b/slideshow/source/engine/slide/layer.cxx @@ -226,7 +226,7 @@ namespace slideshow class LayerEndUpdate : private boost::noncopyable { public: - LayerEndUpdate( LayerSharedPtr const& rLayer ) : + explicit LayerEndUpdate( LayerSharedPtr const& rLayer ) : mpLayer( rLayer ) {} diff --git a/slideshow/source/engine/slideshowimpl.cxx b/slideshow/source/engine/slideshowimpl.cxx index fecd0646c046..bfb8879eaa04 100644 --- a/slideshow/source/engine/slideshowimpl.cxx +++ b/slideshow/source/engine/slideshowimpl.cxx @@ -127,7 +127,7 @@ public: The preferred duration between the display of two frames in seconds. */ - FrameSynchronization (const double nFrameDuration); + explicit FrameSynchronization (const double nFrameDuration); /** Set the current time as the time at which the current frame is displayed. From this the target time of the next frame is derived. @@ -407,7 +407,7 @@ private: void releaseWaitSymbol(); class WaitSymbolLock {public: - WaitSymbolLock(SlideShowImpl& rSlideShowImpl) : mrSlideShowImpl(rSlideShowImpl) + explicit WaitSymbolLock(SlideShowImpl& rSlideShowImpl) : mrSlideShowImpl(rSlideShowImpl) { mrSlideShowImpl.requestWaitSymbol(); } ~WaitSymbolLock() { mrSlideShowImpl.releaseWaitSymbol(); } diff --git a/slideshow/source/engine/smilfunctionparser.cxx b/slideshow/source/engine/smilfunctionparser.cxx index 5b1922de91d0..7e184030cf9d 100644 --- a/slideshow/source/engine/smilfunctionparser.cxx +++ b/slideshow/source/engine/smilfunctionparser.cxx @@ -146,7 +146,7 @@ namespace slideshow class DoubleConstantFunctor { public: - DoubleConstantFunctor( const ParserContextSharedPtr& rContext ) : + explicit DoubleConstantFunctor( const ParserContextSharedPtr& rContext ) : mpContext( rContext ) { ENSURE_OR_THROW( mpContext, @@ -169,7 +169,7 @@ namespace slideshow class ValueTFunctor { public: - ValueTFunctor( const ParserContextSharedPtr& rContext ) : + explicit ValueTFunctor( const ParserContextSharedPtr& rContext ) : mpContext( rContext ) { ENSURE_OR_THROW( mpContext, @@ -413,7 +413,7 @@ namespace slideshow @param rParserContext Contains context info for the parser */ - ExpressionGrammar( const ParserContextSharedPtr& rParserContext ) : + explicit ExpressionGrammar( const ParserContextSharedPtr& rParserContext ) : mpParserContext( rParserContext ) { } @@ -422,7 +422,7 @@ namespace slideshow { public: // grammar definition - definition( const ExpressionGrammar& self ) + explicit definition( const ExpressionGrammar& self ) { using ::boost::spirit::str_p; using ::boost::spirit::real_parser; diff --git a/slideshow/source/engine/tools.cxx b/slideshow/source/engine/tools.cxx index ca98a899fc8e..843a8d273c27 100644 --- a/slideshow/source/engine/tools.cxx +++ b/slideshow/source/engine/tools.cxx @@ -61,7 +61,7 @@ namespace slideshow class NamedValueComparator { public: - NamedValueComparator( const beans::NamedValue& rKey ) : + explicit NamedValueComparator( const beans::NamedValue& rKey ) : mrKey( rKey ) { } diff --git a/slideshow/source/engine/transitions/barndoorwipe.hxx b/slideshow/source/engine/transitions/barndoorwipe.hxx index 71723a72b748..5bd985ab17cd 100644 --- a/slideshow/source/engine/transitions/barndoorwipe.hxx +++ b/slideshow/source/engine/transitions/barndoorwipe.hxx @@ -33,7 +33,7 @@ namespace internal { class BarnDoorWipe : public ParametricPolyPolygon { public: - BarnDoorWipe( bool doubled = false ) + explicit BarnDoorWipe( bool doubled = false ) : m_unitRect( createUnitRect() ), m_doubled(doubled) {} virtual ::basegfx::B2DPolyPolygon operator()( double x ) SAL_OVERRIDE; private: diff --git a/slideshow/source/engine/transitions/barwipepolypolygon.hxx b/slideshow/source/engine/transitions/barwipepolypolygon.hxx index efee2a1a9903..6122c1a800a9 100644 --- a/slideshow/source/engine/transitions/barwipepolypolygon.hxx +++ b/slideshow/source/engine/transitions/barwipepolypolygon.hxx @@ -31,7 +31,7 @@ namespace internal { class BarWipePolyPolygon : public ParametricPolyPolygon { public: - BarWipePolyPolygon( sal_Int32 nBars = 1 /* nBars > 1: blinds effect */ ) + explicit BarWipePolyPolygon( sal_Int32 nBars = 1 /* nBars > 1: blinds effect */ ) : m_nBars(nBars), m_unitRect( createUnitRect() ) {} diff --git a/slideshow/source/engine/transitions/boxwipe.hxx b/slideshow/source/engine/transitions/boxwipe.hxx index 51e111d1b9a1..f72c42c2f49a 100644 --- a/slideshow/source/engine/transitions/boxwipe.hxx +++ b/slideshow/source/engine/transitions/boxwipe.hxx @@ -31,7 +31,7 @@ namespace internal { class BoxWipe : public ParametricPolyPolygon { public: - BoxWipe( bool topCentered ) : m_topCentered(topCentered), + explicit BoxWipe( bool topCentered ) : m_topCentered(topCentered), m_unitRect( createUnitRect() ) {} virtual ::basegfx::B2DPolyPolygon operator () ( double t ) SAL_OVERRIDE; diff --git a/slideshow/source/engine/transitions/checkerboardwipe.hxx b/slideshow/source/engine/transitions/checkerboardwipe.hxx index 384b11e37947..14057568c2f3 100644 --- a/slideshow/source/engine/transitions/checkerboardwipe.hxx +++ b/slideshow/source/engine/transitions/checkerboardwipe.hxx @@ -33,7 +33,7 @@ namespace internal { class CheckerBoardWipe : public ParametricPolyPolygon { public: - CheckerBoardWipe( sal_Int32 unitsPerEdge = 10 ) + explicit CheckerBoardWipe( sal_Int32 unitsPerEdge = 10 ) : m_unitsPerEdge(unitsPerEdge), m_unitRect( createUnitRect() ) { OSL_ASSERT( (unitsPerEdge % 2) == 0 ); } diff --git a/slideshow/source/engine/transitions/ellipsewipe.hxx b/slideshow/source/engine/transitions/ellipsewipe.hxx index 8e69f44fdb17..95f9eb423a59 100644 --- a/slideshow/source/engine/transitions/ellipsewipe.hxx +++ b/slideshow/source/engine/transitions/ellipsewipe.hxx @@ -30,7 +30,7 @@ namespace internal { class EllipseWipe : public ParametricPolyPolygon { public: - EllipseWipe( sal_Int32 /*nTransitionSubType xxx todo */ ) {} + explicit EllipseWipe( sal_Int32 /*nTransitionSubType xxx todo */ ) {} virtual ::basegfx::B2DPolyPolygon operator () ( double x ) SAL_OVERRIDE; }; diff --git a/slideshow/source/engine/transitions/figurewipe.hxx b/slideshow/source/engine/transitions/figurewipe.hxx index c7465917c05d..5f0cdf48c168 100644 --- a/slideshow/source/engine/transitions/figurewipe.hxx +++ b/slideshow/source/engine/transitions/figurewipe.hxx @@ -37,7 +37,7 @@ public: virtual ::basegfx::B2DPolyPolygon operator () ( double t ) SAL_OVERRIDE; private: - FigureWipe( ::basegfx::B2DPolygon const & figure ) : m_figure(figure) {} + explicit FigureWipe( ::basegfx::B2DPolygon const & figure ) : m_figure(figure) {} const ::basegfx::B2DPolygon m_figure; }; diff --git a/slideshow/source/engine/transitions/fourboxwipe.hxx b/slideshow/source/engine/transitions/fourboxwipe.hxx index 780d3d2caaf5..d89bd4f9a858 100644 --- a/slideshow/source/engine/transitions/fourboxwipe.hxx +++ b/slideshow/source/engine/transitions/fourboxwipe.hxx @@ -32,7 +32,7 @@ namespace internal { class FourBoxWipe : public ParametricPolyPolygon { public: - FourBoxWipe( bool cornersOut ) : m_cornersOut(cornersOut), + explicit FourBoxWipe( bool cornersOut ) : m_cornersOut(cornersOut), m_unitRect( createUnitRect() ) {} virtual ::basegfx::B2DPolyPolygon operator () ( double t ) SAL_OVERRIDE; diff --git a/slideshow/source/engine/transitions/pinwheelwipe.hxx b/slideshow/source/engine/transitions/pinwheelwipe.hxx index ff379e0284f2..461e5ded9832 100644 --- a/slideshow/source/engine/transitions/pinwheelwipe.hxx +++ b/slideshow/source/engine/transitions/pinwheelwipe.hxx @@ -31,7 +31,7 @@ namespace internal { class PinWheelWipe : public ParametricPolyPolygon { public: - PinWheelWipe( sal_Int32 blades ) : m_blades(blades) {} + explicit PinWheelWipe( sal_Int32 blades ) : m_blades(blades) {} virtual ::basegfx::B2DPolyPolygon operator () ( double t ) SAL_OVERRIDE; private: sal_Int32 m_blades; diff --git a/slideshow/source/engine/transitions/zigzagwipe.hxx b/slideshow/source/engine/transitions/zigzagwipe.hxx index 30350b39a2f7..5258d2974b64 100644 --- a/slideshow/source/engine/transitions/zigzagwipe.hxx +++ b/slideshow/source/engine/transitions/zigzagwipe.hxx @@ -31,7 +31,7 @@ namespace internal { class ZigZagWipe : public ParametricPolyPolygon { public: - ZigZagWipe( sal_Int32 nZigs ); + explicit ZigZagWipe( sal_Int32 nZigs ); virtual ::basegfx::B2DPolyPolygon operator () ( double t ) SAL_OVERRIDE; protected: const double m_zigEdge; @@ -42,7 +42,7 @@ protected: class BarnZigZagWipe : public ZigZagWipe { public: - BarnZigZagWipe( sal_Int32 nZigs ) : ZigZagWipe(nZigs) {} + explicit BarnZigZagWipe( sal_Int32 nZigs ) : ZigZagWipe(nZigs) {} virtual ::basegfx::B2DPolyPolygon operator () ( double t ) SAL_OVERRIDE; }; diff --git a/slideshow/source/engine/usereventqueue.cxx b/slideshow/source/engine/usereventqueue.cxx index 9f06797b78ea..ab31873c83cf 100644 --- a/slideshow/source/engine/usereventqueue.cxx +++ b/slideshow/source/engine/usereventqueue.cxx @@ -123,7 +123,7 @@ protected: class AllAnimationEventHandler : public AnimationEventHandler { public: - AllAnimationEventHandler( EventQueue& rEventQueue ) : + explicit AllAnimationEventHandler( EventQueue& rEventQueue ) : mrEventQueue( rEventQueue ), maAnimationEventMap() {} @@ -182,7 +182,7 @@ class ClickEventHandler : public MouseEventHandler_, public EventContainer { public: - ClickEventHandler( EventQueue& rEventQueue ) : + explicit ClickEventHandler( EventQueue& rEventQueue ) : EventContainer(), mrEventQueue( rEventQueue ), mbAdvanceOnClick( true ) @@ -293,7 +293,7 @@ private: class MouseHandlerBase : public MouseEventHandler_ { public: - MouseHandlerBase( EventQueue& rEventQueue ) : + explicit MouseHandlerBase( EventQueue& rEventQueue ) : mrEventQueue( rEventQueue ), maShapeEventMap() {} @@ -422,7 +422,7 @@ private: class MouseEnterHandler : public MouseHandlerBase { public: - MouseEnterHandler( EventQueue& rEventQueue ) + explicit MouseEnterHandler( EventQueue& rEventQueue ) : MouseHandlerBase( rEventQueue ), mpLastShape() {} @@ -460,7 +460,7 @@ private: class MouseLeaveHandler : public MouseHandlerBase { public: - MouseLeaveHandler( EventQueue& rEventQueue ) + explicit MouseLeaveHandler( EventQueue& rEventQueue ) : MouseHandlerBase( rEventQueue ), maLastIter() {} |