[Libreoffice-commits] core.git: 2 commits - chart2/source include/svx sw/source

Caolán McNamara caolanm at redhat.com
Wed Jul 2 06:02:26 PDT 2014


 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 <caolanm at 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 <caolanm at 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


More information about the Libreoffice-commits mailing list