chart2/source/controller/chartapiwrapper/WrappedStatisticProperties.cxx | 33 ++--- chart2/source/controller/itemsetwrapper/RegressionCurveItemConverter.cxx | 39 ------ chart2/source/controller/itemsetwrapper/StatisticsItemConverter.cxx | 57 +--------- chart2/source/controller/main/ChartController_Insert.cxx | 2 chart2/source/inc/RegressionCurveHelper.hxx | 22 --- chart2/source/tools/RegressionCurveHelper.cxx | 48 ++++---- include/svx/chrtitem.hxx | 7 - sw/source/core/layout/atrfrm.cxx | 40 ++++++- 8 files changed, 95 insertions(+), 153 deletions(-)
New commits: commit 2b0df7420699e77d8a6e2f9e758939743e94986d Author: Caolán McNamara <caol...@redhat.com> Date: Wed Jul 2 13:54:11 2014 +0100 coverity#705381 Mixing enum types Change-Id: I1dad21f91f5195585ca9125b7f8f47c7bd5a1a76 diff --git a/sw/source/core/layout/atrfrm.cxx b/sw/source/core/layout/atrfrm.cxx index 63ca034..3237455 100644 --- a/sw/source/core/layout/atrfrm.cxx +++ b/sw/source/core/layout/atrfrm.cxx @@ -1126,6 +1126,36 @@ sal_uInt16 SwFmtSurround::GetValueCount() const return SURROUND_END - SURROUND_BEGIN; } +namespace +{ + text::WrapTextMode SwSurroundToWrapMode(SwSurround eSurround) + { + text::WrapTextMode eRet; + switch(eSurround) + { + case SURROUND_THROUGHT: + eRet = css::text::WrapTextMode_THROUGHT; + break; + case SURROUND_PARALLEL: + eRet = css::text::WrapTextMode_PARALLEL; + break; + case SURROUND_IDEAL: + eRet = css::text::WrapTextMode_DYNAMIC; + break; + case SURROUND_LEFT: + eRet = css::text::WrapTextMode_LEFT; + break; + case SURROUND_RIGHT: + eRet = css::text::WrapTextMode_RIGHT; + break; + default: + eRet = css::text::WrapTextMode_NONE; + break; + } + return eRet; + } +} + bool SwFmtSurround::QueryValue( uno::Any& rVal, sal_uInt8 nMemberId ) const { // here we convert always! @@ -1134,26 +1164,26 @@ bool SwFmtSurround::QueryValue( uno::Any& rVal, sal_uInt8 nMemberId ) const switch ( nMemberId ) { case MID_SURROUND_SURROUNDTYPE: - rVal <<= (text::WrapTextMode)GetSurround(); - break; + rVal <<= SwSurroundToWrapMode(GetSurround()); + break; case MID_SURROUND_ANCHORONLY: { bool bTmp = IsAnchorOnly(); rVal.setValue(&bTmp, ::getBooleanCppuType()); + break; } - break; case MID_SURROUND_CONTOUR: { bool bTmp = IsContour(); rVal.setValue(&bTmp, ::getBooleanCppuType()); + break; } - break; case MID_SURROUND_CONTOUROUTSIDE: { bool bTmp = IsOutside(); rVal.setValue(&bTmp, ::getBooleanCppuType()); + break; } - break; default: OSL_ENSURE( !this, "unknown MemberId" ); bRet = false; commit 8918dabbeed24d68a746309165e6c09a09b7b2f3 Author: Caolán McNamara <caol...@redhat.com> Date: Wed Jul 2 13:42:16 2014 +0100 coverity#705359 Mixing enum types and coverity#705361 Mixing enum types coverity#705362 Mixing enum types Change-Id: I557d7178277e5d497e3bbab12b75724c05653f57 diff --git a/chart2/source/controller/chartapiwrapper/WrappedStatisticProperties.cxx b/chart2/source/controller/chartapiwrapper/WrappedStatisticProperties.cxx index 8dd0f80..1c63fe6 100644 --- a/chart2/source/controller/chartapiwrapper/WrappedStatisticProperties.cxx +++ b/chart2/source/controller/chartapiwrapper/WrappedStatisticProperties.cxx @@ -56,27 +56,27 @@ Any lcl_getRegressionDefault() return aRet; } -::com::sun::star::chart::ChartRegressionCurveType lcl_getRegressionCurveType( RegressionCurveHelper::tRegressionType eRegressionType ) +::com::sun::star::chart::ChartRegressionCurveType lcl_getRegressionCurveType(SvxChartRegress eRegressionType) { ::com::sun::star::chart::ChartRegressionCurveType eRet = ::com::sun::star::chart::ChartRegressionCurveType_NONE; switch(eRegressionType) { - case RegressionCurveHelper::REGRESSION_TYPE_LINEAR: + case CHREGRESS_LINEAR: eRet = ::com::sun::star::chart::ChartRegressionCurveType_LINEAR; break; - case RegressionCurveHelper::REGRESSION_TYPE_LOG: + case CHREGRESS_LOG: eRet = ::com::sun::star::chart::ChartRegressionCurveType_LOGARITHM; break; - case RegressionCurveHelper::REGRESSION_TYPE_EXP: + case CHREGRESS_EXP: eRet = ::com::sun::star::chart::ChartRegressionCurveType_EXPONENTIAL; break; - case RegressionCurveHelper::REGRESSION_TYPE_POWER: + case CHREGRESS_POWER: eRet = ::com::sun::star::chart::ChartRegressionCurveType_POWER; break; - case RegressionCurveHelper::REGRESSION_TYPE_POLYNOMIAL: + case CHREGRESS_POLYNOMIAL: eRet = ::com::sun::star::chart::ChartRegressionCurveType_POLYNOMIAL; break; - /*case RegressionCurveHelper::REGRESSION_TYPE_MOVING_AVERAGE: + /*case CHREGRESS_MOVING_AVERAGE: eRet = ::com::sun::star::chart::ChartRegressionCurveType_MOVING_AVERAGE; break;*/ default: @@ -86,27 +86,27 @@ Any lcl_getRegressionDefault() return eRet; } -RegressionCurveHelper::tRegressionType lcl_getRegressionType( ::com::sun::star::chart::ChartRegressionCurveType eRegressionCurveType ) +SvxChartRegress lcl_getRegressionType( ::com::sun::star::chart::ChartRegressionCurveType eRegressionCurveType ) { - RegressionCurveHelper::tRegressionType eRet; - switch(eRegressionCurveType) + SvxChartRegress eRet; + switch (eRegressionCurveType) { case ::com::sun::star::chart::ChartRegressionCurveType_LINEAR: - eRet = RegressionCurveHelper::REGRESSION_TYPE_LINEAR; + eRet = CHREGRESS_LINEAR; break; case ::com::sun::star::chart::ChartRegressionCurveType_LOGARITHM: - eRet = RegressionCurveHelper::REGRESSION_TYPE_LOG; + eRet = CHREGRESS_LOG; break; case ::com::sun::star::chart::ChartRegressionCurveType_EXPONENTIAL: - eRet = RegressionCurveHelper::REGRESSION_TYPE_EXP; + eRet = CHREGRESS_EXP; break; case ::com::sun::star::chart::ChartRegressionCurveType_POLYNOMIAL: //case ::com::sun::star::chart::ChartRegressionCurveType_MOVING_AVERAGE: case ::com::sun::star::chart::ChartRegressionCurveType_POWER: - eRet = RegressionCurveHelper::REGRESSION_TYPE_POWER; + eRet = CHREGRESS_POWER; break; default: - eRet = RegressionCurveHelper::REGRESSION_TYPE_NONE; + eRet = CHREGRESS_NONE; break; } return eRet; @@ -831,6 +831,7 @@ WrappedRegressionCurvesProperty::WrappedRegressionCurvesProperty( , lcl_getRegressionDefault(), spChart2ModelContact, ePropertyType ) { } + WrappedRegressionCurvesProperty::~WrappedRegressionCurvesProperty() { } @@ -854,7 +855,7 @@ void WrappedRegressionCurvesProperty::setValueToSeries( const Reference< beans:: if( xRegressionCurveContainer.is() && xRegressionCurve.is() ) { - RegressionCurveHelper::tRegressionType eNewRegressionType = lcl_getRegressionType( aNewValue ); + SvxChartRegress eNewRegressionType = lcl_getRegressionType( aNewValue ); RegressionCurveHelper::changeRegressionCurveType( eNewRegressionType, diff --git a/chart2/source/controller/itemsetwrapper/RegressionCurveItemConverter.cxx b/chart2/source/controller/itemsetwrapper/RegressionCurveItemConverter.cxx index df4b19b..fc2cc14 100644 --- a/chart2/source/controller/itemsetwrapper/RegressionCurveItemConverter.cxx +++ b/chart2/source/controller/itemsetwrapper/RegressionCurveItemConverter.cxx @@ -29,7 +29,6 @@ #include <svl/eitem.hxx> #include <svl/intitem.hxx> #include <svl/stritem.hxx> -#include <svx/chrtitem.hxx> #include <functional> #include <algorithm> @@ -38,36 +37,6 @@ using namespace ::com::sun::star; namespace { - -::chart::RegressionCurveHelper::tRegressionType lcl_convertRegressionType( SvxChartRegress eRegress ) -{ - ::chart::RegressionCurveHelper::tRegressionType eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_NONE; - switch( eRegress ) - { - case CHREGRESS_LINEAR: - eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_LINEAR; - break; - case CHREGRESS_LOG: - eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_LOG; - break; - case CHREGRESS_EXP: - eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_EXP; - break; - case CHREGRESS_POWER: - eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_POWER; - break; - case CHREGRESS_POLYNOMIAL: - eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_POLYNOMIAL; - break; - case CHREGRESS_MOVING_AVERAGE: - eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_MOVING_AVERAGE; - break; - case CHREGRESS_NONE: - break; - } - return eType; -} - template <class T, class D> bool lclConvertToPropertySet(const SfxItemSet& rItemSet, sal_uInt16 nWhichId, uno::Reference<beans::XPropertySet> xProperties, const OUString& aPropertyID) { @@ -181,8 +150,7 @@ bool RegressionCurveItemConverter::ApplySpecialItem( { case SCHATTR_REGRESSION_TYPE: { - SvxChartRegress eRegress = static_cast< SvxChartRegress >( - static_cast< sal_Int32 >( RegressionCurveHelper::getRegressionType( xCurve ))); + SvxChartRegress eRegress = RegressionCurveHelper::getRegressionType(xCurve); SvxChartRegress eNewRegress = static_cast< const SvxChartRegressItem & >( rItemSet.Get( nWhichId )).GetValue(); if( eRegress != eNewRegress ) @@ -192,7 +160,7 @@ bool RegressionCurveItemConverter::ApplySpecialItem( // currently the only way to handle the type in the // regression curve properties dialog xCurve = RegressionCurveHelper::changeRegressionCurveType( - lcl_convertRegressionType( eNewRegress ), + eNewRegress, m_xCurveContainer, xCurve, uno::Reference< uno::XComponentContext >()); @@ -284,8 +252,7 @@ void RegressionCurveItemConverter::FillSpecialItem(sal_uInt16 nWhichId, SfxItemS { case SCHATTR_REGRESSION_TYPE: { - sal_Int32 aRegressionType = static_cast< sal_Int32 >(RegressionCurveHelper::getRegressionType(xCurve)); - SvxChartRegress eRegress = static_cast< SvxChartRegress >(aRegressionType); + SvxChartRegress eRegress = RegressionCurveHelper::getRegressionType(xCurve); rOutItemSet.Put( SvxChartRegressItem( eRegress, SCHATTR_REGRESSION_TYPE )); } break; diff --git a/chart2/source/controller/itemsetwrapper/StatisticsItemConverter.cxx b/chart2/source/controller/itemsetwrapper/StatisticsItemConverter.cxx index 1f24e8b..a7a0b5c 100644 --- a/chart2/source/controller/itemsetwrapper/StatisticsItemConverter.cxx +++ b/chart2/source/controller/itemsetwrapper/StatisticsItemConverter.cxx @@ -68,35 +68,6 @@ uno::Reference< beans::XPropertySet > lcl_GetErrorBar( return xResult; } -::chart::RegressionCurveHelper::tRegressionType lcl_convertRegressionType( SvxChartRegress eRegress ) -{ - ::chart::RegressionCurveHelper::tRegressionType eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_NONE; - switch( eRegress ) - { - case CHREGRESS_LINEAR: - eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_LINEAR; - break; - case CHREGRESS_LOG: - eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_LOG; - break; - case CHREGRESS_EXP: - eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_EXP; - break; - case CHREGRESS_POWER: - eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_POWER; - break; - case CHREGRESS_POLYNOMIAL: - eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_POLYNOMIAL; - break; - case CHREGRESS_MOVING_AVERAGE: - eType = ::chart::RegressionCurveHelper::REGRESSION_TYPE_MOVING_AVERAGE; - break; - case CHREGRESS_NONE: - break; - } - return eType; -} - uno::Reference< beans::XPropertySet > lcl_GetDefaultErrorBar() { // todo: use a valid context @@ -262,24 +233,11 @@ StatisticsItemConverter::StatisticsItemConverter( ItemConverter( rPropertySet, rItemPool ), m_xModel( xModel ) { - OSL_ASSERT( static_cast< int >( RegressionCurveHelper::REGRESSION_TYPE_NONE ) == - static_cast< int >( CHREGRESS_NONE )); - OSL_ASSERT( static_cast< int >( RegressionCurveHelper::REGRESSION_TYPE_LINEAR ) == - static_cast< int >( CHREGRESS_LINEAR )); - OSL_ASSERT( static_cast< int >( RegressionCurveHelper::REGRESSION_TYPE_LOG ) == - static_cast< int >( CHREGRESS_LOG )); - OSL_ASSERT( static_cast< int >( RegressionCurveHelper::REGRESSION_TYPE_EXP ) == - static_cast< int >( CHREGRESS_EXP )); - OSL_ASSERT( static_cast< int >( RegressionCurveHelper::REGRESSION_TYPE_POWER ) == - static_cast< int >( CHREGRESS_POWER )); - OSL_ASSERT( static_cast< int >( RegressionCurveHelper::REGRESSION_TYPE_POLYNOMIAL ) == - static_cast< int >( CHREGRESS_POLYNOMIAL )); - OSL_ASSERT( static_cast< int >( RegressionCurveHelper::REGRESSION_TYPE_MOVING_AVERAGE ) == - static_cast< int >( CHREGRESS_MOVING_AVERAGE )); } StatisticsItemConverter::~StatisticsItemConverter() -{} +{ +} const sal_uInt16 * StatisticsItemConverter::GetWhichPairs() const { @@ -476,14 +434,12 @@ bool StatisticsItemConverter::ApplySpecialItem( if ( xCurve.is() ) { SvxChartRegress eOldRegress( - static_cast< SvxChartRegress >( - static_cast< sal_Int32 >( - RegressionCurveHelper::getRegressionType( xCurve )))); + RegressionCurveHelper::getRegressionType(xCurve)); if( eOldRegress != eRegress ) { xCurve = RegressionCurveHelper::changeRegressionCurveType( - lcl_convertRegressionType( eRegress ), + eRegress, xContainer, xCurve, uno::Reference< uno::XComponentContext >()); @@ -756,11 +712,10 @@ void StatisticsItemConverter::FillSpecialItem( case SCHATTR_REGRESSION_TYPE: { - SvxChartRegress eRegress = static_cast< SvxChartRegress >( - static_cast< sal_Int32 >( + SvxChartRegress eRegress = RegressionCurveHelper::getFirstRegressTypeNotMeanValueLine( uno::Reference< chart2::XRegressionCurveContainer >( - GetPropertySet(), uno::UNO_QUERY ) ))); + GetPropertySet(), uno::UNO_QUERY ) ); rOutItemSet.Put( SvxChartRegressItem( eRegress, SCHATTR_REGRESSION_TYPE )); } break; diff --git a/chart2/source/controller/main/ChartController_Insert.cxx b/chart2/source/controller/main/ChartController_Insert.cxx index 610bfae..1111bbe 100644 --- a/chart2/source/controller/main/ChartController_Insert.cxx +++ b/chart2/source/controller/main/ChartController_Insert.cxx @@ -373,7 +373,7 @@ void ChartController::executeDispatch_InsertTrendline() uno::Reference< chart2::XRegressionCurve > xCurve = RegressionCurveHelper::addRegressionCurve( - RegressionCurveHelper::REGRESSION_TYPE_LINEAR, + CHREGRESS_LINEAR, xRegressionCurveContainer, m_xCC ); diff --git a/chart2/source/inc/RegressionCurveHelper.hxx b/chart2/source/inc/RegressionCurveHelper.hxx index ee1d160..772c3c3 100644 --- a/chart2/source/inc/RegressionCurveHelper.hxx +++ b/chart2/source/inc/RegressionCurveHelper.hxx @@ -27,6 +27,7 @@ #include <com/sun/star/chart2/XDiagram.hpp> #include <com/sun/star/uno/XComponentContext.hpp> #include <com/sun/star/frame/XModel.hpp> +#include <svx/chrtitem.hxx> #include "charttoolsdllapi.hxx" #include <vector> @@ -70,19 +71,6 @@ public: static void removeMeanValueLine( css::uno::Reference<css::chart2::XRegressionCurveContainer>& xRegCnt ); - enum tRegressionType - { - REGRESSION_TYPE_NONE, - REGRESSION_TYPE_LINEAR, - REGRESSION_TYPE_LOG, - REGRESSION_TYPE_EXP, - REGRESSION_TYPE_POWER, - REGRESSION_TYPE_POLYNOMIAL, - REGRESSION_TYPE_MOVING_AVERAGE, - REGRESSION_TYPE_MEAN_VALUE, - REGRESSION_TYPE_UNKNOWN - }; - /** Returns the first regression curve found that is not of type mean-value line */ @@ -100,10 +88,10 @@ public: /** Returns the type of the first regression curve found that is not of type mean-value line */ - static tRegressionType getFirstRegressTypeNotMeanValueLine( + static SvxChartRegress getFirstRegressTypeNotMeanValueLine( const css::uno::Reference<css::chart2::XRegressionCurveContainer>& xCurveContainer ); - static tRegressionType getRegressionType( + static SvxChartRegress getRegressionType( const css::uno::Reference<css::chart2::XRegressionCurve>& xCurve ); /** @param xPropertySource is taken as source to copy all properties from if @@ -113,7 +101,7 @@ public: */ static css::uno::Reference<css::chart2::XRegressionCurve> addRegressionCurve( - tRegressionType eType, + SvxChartRegress eType, css::uno::Reference<css::chart2::XRegressionCurveContainer>& xCurveContainer, const css::uno::Reference<css::uno::XComponentContext>& xContext, const css::uno::Reference<css::beans::XPropertySet >& xPropertySource = @@ -129,7 +117,7 @@ public: static css::uno::Reference<css::chart2::XRegressionCurve> changeRegressionCurveType( - tRegressionType eType, + SvxChartRegress eType, css::uno::Reference<css::chart2::XRegressionCurveContainer>& xRegressionCurveContainer, css::uno::Reference<css::chart2::XRegressionCurve>& xRegressionCurve, const css::uno::Reference<css::uno::XComponentContext>& xContext ); diff --git a/chart2/source/tools/RegressionCurveHelper.cxx b/chart2/source/tools/RegressionCurveHelper.cxx index 1f243370..fd3ac69 100644 --- a/chart2/source/tools/RegressionCurveHelper.cxx +++ b/chart2/source/tools/RegressionCurveHelper.cxx @@ -48,27 +48,27 @@ using ::com::sun::star::uno::Exception; namespace { -OUString lcl_getServiceNameForType( ::chart::RegressionCurveHelper::tRegressionType eType ) +OUString lcl_getServiceNameForType(SvxChartRegress eType) { OUString aServiceName; switch( eType ) { - case ::chart::RegressionCurveHelper::REGRESSION_TYPE_LINEAR: + case CHREGRESS_LINEAR: aServiceName = "com.sun.star.chart2.LinearRegressionCurve"; break; - case ::chart::RegressionCurveHelper::REGRESSION_TYPE_LOG: + case CHREGRESS_LOG: aServiceName = "com.sun.star.chart2.LogarithmicRegressionCurve"; break; - case ::chart::RegressionCurveHelper::REGRESSION_TYPE_EXP: + case CHREGRESS_EXP: aServiceName = "com.sun.star.chart2.ExponentialRegressionCurve"; break; - case ::chart::RegressionCurveHelper::REGRESSION_TYPE_POWER: + case CHREGRESS_POWER: aServiceName = "com.sun.star.chart2.PotentialRegressionCurve"; break; - case ::chart::RegressionCurveHelper::REGRESSION_TYPE_POLYNOMIAL: + case CHREGRESS_POLYNOMIAL: aServiceName = "com.sun.star.chart2.PolynomialRegressionCurve"; break; - case ::chart::RegressionCurveHelper::REGRESSION_TYPE_MOVING_AVERAGE: + case CHREGRESS_MOVING_AVERAGE: aServiceName = "com.sun.star.chart2.MovingAverageRegressionCurve"; break; default: @@ -351,7 +351,7 @@ void RegressionCurveHelper::removeMeanValueLine( } uno::Reference< chart2::XRegressionCurve > RegressionCurveHelper::addRegressionCurve( - tRegressionType eType, + SvxChartRegress eType, uno::Reference< XRegressionCurveContainer >& xRegressionCurveContainer, const uno::Reference< XComponentContext >& /* xContext */, const uno::Reference< beans::XPropertySet >& xPropertySource, @@ -362,7 +362,7 @@ uno::Reference< chart2::XRegressionCurve > RegressionCurveHelper::addRegressionC if( !xRegressionCurveContainer.is() ) return xCurve; - if( eType == REGRESSION_TYPE_NONE ) + if( eType == CHREGRESS_NONE ) { OSL_FAIL("don't create a regression curve of type none"); return xCurve; @@ -470,7 +470,7 @@ void RegressionCurveHelper::removeEquations( } uno::Reference< XRegressionCurve > RegressionCurveHelper::changeRegressionCurveType( - tRegressionType eType, + SvxChartRegress eType, uno::Reference< XRegressionCurveContainer > & xRegressionCurveContainer, uno::Reference< XRegressionCurve > & xRegressionCurve, const uno::Reference< XComponentContext > & xContext ) @@ -534,10 +534,10 @@ uno::Reference< chart2::XRegressionCurve > RegressionCurveHelper::getRegressionC return NULL; } -RegressionCurveHelper::tRegressionType RegressionCurveHelper::getRegressionType( +SvxChartRegress RegressionCurveHelper::getRegressionType( const Reference< XRegressionCurve > & xCurve ) { - tRegressionType eResult = REGRESSION_TYPE_UNKNOWN; + SvxChartRegress eResult = CHREGRESS_UNKNOWN; try { @@ -548,31 +548,31 @@ RegressionCurveHelper::tRegressionType RegressionCurveHelper::getRegressionType( if( aServiceName == "com.sun.star.chart2.LinearRegressionCurve" ) { - eResult = REGRESSION_TYPE_LINEAR; + eResult = CHREGRESS_LINEAR; } else if( aServiceName == "com.sun.star.chart2.LogarithmicRegressionCurve" ) { - eResult = REGRESSION_TYPE_LOG; + eResult = CHREGRESS_LOG; } else if( aServiceName == "com.sun.star.chart2.ExponentialRegressionCurve" ) { - eResult = REGRESSION_TYPE_EXP; + eResult = CHREGRESS_EXP; } else if( aServiceName == "com.sun.star.chart2.PotentialRegressionCurve" ) { - eResult = REGRESSION_TYPE_POWER; + eResult = CHREGRESS_POWER; } else if( aServiceName == "com.sun.star.chart2.MeanValueRegressionCurve" ) { - eResult = REGRESSION_TYPE_MEAN_VALUE; + eResult = CHREGRESS_MEAN_VALUE; } else if( aServiceName == "com.sun.star.chart2.PolynomialRegressionCurve" ) { - eResult = REGRESSION_TYPE_POLYNOMIAL; + eResult = CHREGRESS_POLYNOMIAL; } else if( aServiceName == "com.sun.star.chart2.MovingAverageRegressionCurve" ) { - eResult = REGRESSION_TYPE_MOVING_AVERAGE; + eResult = CHREGRESS_MOVING_AVERAGE; } } } @@ -584,10 +584,10 @@ RegressionCurveHelper::tRegressionType RegressionCurveHelper::getRegressionType( return eResult; } -RegressionCurveHelper::tRegressionType RegressionCurveHelper::getFirstRegressTypeNotMeanValueLine( +SvxChartRegress RegressionCurveHelper::getFirstRegressTypeNotMeanValueLine( const Reference< XRegressionCurveContainer > & xRegCnt ) { - tRegressionType eResult = REGRESSION_TYPE_NONE; + SvxChartRegress eResult = CHREGRESS_NONE; if( xRegCnt.is()) { @@ -595,9 +595,9 @@ RegressionCurveHelper::tRegressionType RegressionCurveHelper::getFirstRegressTyp xRegCnt->getRegressionCurves()); for( sal_Int32 i = 0; i < aCurves.getLength(); ++i ) { - tRegressionType eType = getRegressionType( aCurves[i] ); - if( eType != REGRESSION_TYPE_MEAN_VALUE && - eType != REGRESSION_TYPE_UNKNOWN ) + SvxChartRegress eType = getRegressionType( aCurves[i] ); + if( eType != CHREGRESS_MEAN_VALUE && + eType != CHREGRESS_UNKNOWN ) { eResult = eType; break; diff --git a/include/svx/chrtitem.hxx b/include/svx/chrtitem.hxx index 63c57d6..750985b 100644 --- a/include/svx/chrtitem.hxx +++ b/include/svx/chrtitem.hxx @@ -157,11 +157,12 @@ enum SvxChartRegress CHREGRESS_EXP, CHREGRESS_POWER, CHREGRESS_POLYNOMIAL, - CHREGRESS_MOVING_AVERAGE + CHREGRESS_MOVING_AVERAGE, + CHREGRESS_MEAN_VALUE, + CHREGRESS_UNKNOWN }; -#define CHREGRESS_COUNT (CHREGRESS_MOVING_AVERAGE + 1) - +#define CHREGRESS_COUNT (CHREGRESS_UNKNOWN + 1) class SVX_DLLPUBLIC SvxChartStyleItem : public SfxEnumItem
_______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits