From 6f31c63e35abef03e6f938bbddc8778b70a62d43 Mon Sep 17 00:00:00 2001 From: Arkadiy Illarionov Date: Sat, 6 Apr 2019 13:36:35 +0300 Subject: [PATCH] clang-tidy modernize-avoid-c-arrays in animations Change-Id: I3dc1c0e63bd90735c20a65b1af25b243e5a5eee5 Reviewed-on: https://gerrit.libreoffice.org/70341 Tested-by: Jenkins Reviewed-by: Noel Grandin --- animations/source/animcore/animcore.cxx | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/animations/source/animcore/animcore.cxx b/animations/source/animcore/animcore.cxx index 4dd4868e70e5..d5bcb0c21b14 100644 --- a/animations/source/animcore/animcore.cxx +++ b/animations/source/animcore/animcore.cxx @@ -57,6 +57,7 @@ #include #include +#include #include #include #include @@ -287,7 +288,7 @@ private: const sal_Int16 mnNodeType; // for XTypeProvider - static Sequence< Type >* mpTypes[12]; + static std::array*, 12> mpTypes; // attributes for the XAnimationNode interface implementation Any maBegin, maDuration, maEnd, maEndSync, maRepeatCount, maRepeatDuration; @@ -388,7 +389,7 @@ Any SAL_CALL TimeContainerEnumeration::nextElement() } -Sequence< Type >* AnimationNode::mpTypes[] = { nullptr, nullptr, nullptr, nullptr, nullptr, nullptr, nullptr, nullptr, nullptr, nullptr, nullptr }; +std::array*, 12> AnimationNode::mpTypes = { nullptr, nullptr, nullptr, nullptr, nullptr, nullptr, nullptr, nullptr, nullptr, nullptr, nullptr, nullptr }; AnimationNode::AnimationNode( sal_Int16 nNodeType ) : maChangeListener(maMutex), @@ -418,7 +419,7 @@ AnimationNode::AnimationNode( sal_Int16 nNodeType ) mnIterateType( css::presentation::ShapeAnimationSubType::AS_WHOLE ), mfIterateInterval(0.0) { - assert(nNodeType < int(SAL_N_ELEMENTS(mpTypes))); + assert(nNodeType < int(mpTypes.size())); } AnimationNode::AnimationNode( const AnimationNode& rNode ) @@ -696,7 +697,7 @@ void AnimationNode::initTypeProvider( sal_Int16 nNodeType ) throw() if(! mpTypes[nNodeType] ) { - static const sal_Int32 type_numbers[] = + static constexpr std::array type_numbers = { 7, // CUSTOM 9, // PAR