[Libreoffice-commits] core.git: editeng/source include/editeng sw/source

Noel Grandin noel at peralex.com
Mon Aug 15 11:54:31 UTC 2016


 editeng/source/editeng/edtspell.cxx  |    6 +++---
 editeng/source/editeng/textconv.cxx  |   14 +++++++-------
 editeng/source/misc/splwrap.cxx      |   14 +++++++-------
 include/editeng/svxenum.hxx          |   10 +++++-----
 sw/source/uibase/lingu/hhcwrp.cxx    |   12 ++++++------
 sw/source/uibase/lingu/hyp.cxx       |    2 +-
 sw/source/uibase/uiview/viewling.cxx |   16 ++++++++--------
 7 files changed, 37 insertions(+), 37 deletions(-)

New commits:
commit 6f6024610b6f5392a96ed07ce9abe5aab606519d
Author: Noel Grandin <noel at peralex.com>
Date:   Mon Aug 15 13:53:33 2016 +0200

    convert SvxSpellArea to scoped enum
    
    Change-Id: I91d3caabb667e7d4ff23e603026e072074058712

diff --git a/editeng/source/editeng/edtspell.cxx b/editeng/source/editeng/edtspell.cxx
index 627ead7..4077ae5 100644
--- a/editeng/source/editeng/edtspell.cxx
+++ b/editeng/source/editeng/edtspell.cxx
@@ -57,7 +57,7 @@ void EditSpellWrapper::SpellStart( SvxSpellArea eArea )
     ImpEditEngine* pImpEE = pEditView->GetImpEditEngine();
     SpellInfo* pSpellInfo = pImpEE->GetSpellInfo();
 
-    if ( eArea == SVX_SPELL_BODY_START )
+    if ( eArea == SvxSpellArea::BodyStart )
     {
         // Is called when
         // a) Spell-Forward has arrived at the end and should restart at the top
@@ -76,7 +76,7 @@ void EditSpellWrapper::SpellStart( SvxSpellArea eArea )
                     pEE->GetEditDoc().GetStartPaM() );
         }
     }
-    else if ( eArea == SVX_SPELL_BODY_END )
+    else if ( eArea == SvxSpellArea::BodyEnd )
     {
         // Is called when
         // a) Spell-Forward is launched
@@ -95,7 +95,7 @@ void EditSpellWrapper::SpellStart( SvxSpellArea eArea )
                     pEE->GetEditDoc().GetEndPaM() );
         }
     }
-    else if ( eArea == SVX_SPELL_BODY )
+    else if ( eArea == SvxSpellArea::Body )
     {
         ;   // Is handled by the App through SpellNextDocument
     }
diff --git a/editeng/source/editeng/textconv.cxx b/editeng/source/editeng/textconv.cxx
index fd73733..7a8d6db 100644
--- a/editeng/source/editeng/textconv.cxx
+++ b/editeng/source/editeng/textconv.cxx
@@ -87,7 +87,7 @@ bool TextConvWrapper::ConvNext_impl()
         {
             m_bStartDone = true;
             m_bEndDone  = false;
-            ConvStart_impl( SVX_SPELL_BODY );
+            ConvStart_impl( SvxSpellArea::Body );
             return true;
         }
         return false;
@@ -100,14 +100,14 @@ bool TextConvWrapper::ConvNext_impl()
         {
             m_bStartDone = true;
             m_bEndDone  = false;
-            ConvStart_impl( SVX_SPELL_BODY );
+            ConvStart_impl( SvxSpellArea::Body );
             return true;
         }
     }
     else if (!m_aConvSel.HasRange())
     {
         m_bStartChk = !m_bStartDone;
-        ConvStart_impl( m_bStartChk ? SVX_SPELL_BODY_START : SVX_SPELL_BODY_END );
+        ConvStart_impl( m_bStartChk ? SvxSpellArea::BodyStart : SvxSpellArea::BodyEnd );
         return true;
     }
 
@@ -160,7 +160,7 @@ void TextConvWrapper::ConvStart_impl( SvxSpellArea eArea )
     ImpEditEngine* pImpEE = m_pEditView->GetImpEditEngine();
     ConvInfo* pConvInfo = pImpEE->GetConvInfo();
 
-    if ( eArea == SVX_SPELL_BODY_START )
+    if ( eArea == SvxSpellArea::BodyStart )
     {
         // Is called when Spell-forward has reached the end, and to start over
         if ( m_bEndDone )
@@ -178,7 +178,7 @@ void TextConvWrapper::ConvStart_impl( SvxSpellArea eArea )
                     pEE->GetEditDoc().GetStartPaM() );
         }
     }
-    else if ( eArea == SVX_SPELL_BODY_END )
+    else if ( eArea == SvxSpellArea::BodyEnd )
     {
         // Is called when Spell-forward starts
         pConvInfo->bConvToEnd = true;
@@ -196,7 +196,7 @@ void TextConvWrapper::ConvStart_impl( SvxSpellArea eArea )
                 pEE->GetEditDoc().GetEndPaM() );
         }
     }
-    else if ( eArea == SVX_SPELL_BODY )
+    else if ( eArea == SvxSpellArea::Body )
     {
         // called by ConvNext_impl...
         pConvInfo->aConvContinue = pConvInfo->aConvStart;
@@ -542,7 +542,7 @@ void TextConvWrapper::ChangeText_impl( const OUString &rNewText, bool bKeepAttri
 void TextConvWrapper::Convert()
 {
     m_bStartChk = false;
-    ConvStart_impl( SVX_SPELL_BODY_END );
+    ConvStart_impl( SvxSpellArea::BodyEnd );
     ConvertDocument();
 }
 
diff --git a/editeng/source/misc/splwrap.cxx b/editeng/source/misc/splwrap.cxx
index c8f0769..35c68be 100644
--- a/editeng/source/misc/splwrap.cxx
+++ b/editeng/source/misc/splwrap.cxx
@@ -275,12 +275,12 @@ void SvxSpellWrapper::SpellDocument( )
     if ( bOtherCntnt )
     {
         bReverse = false;
-        SpellStart( SVX_SPELL_OTHER );
+        SpellStart( SvxSpellArea::Other );
     }
     else
     {
         bStartChk = bReverse;
-        SpellStart( bReverse ? SVX_SPELL_BODY_START : SVX_SPELL_BODY_END );
+        SpellStart( bReverse ? SvxSpellArea::BodyStart : SvxSpellArea::BodyEnd );
     }
 
     if ( FindSpellError() )
@@ -340,7 +340,7 @@ bool SvxSpellWrapper::SpellNext( )
             bOtherCntnt = false;
             bStartDone = !bReverse;
             bEndDone  = bReverse;
-            SpellStart( SVX_SPELL_BODY );
+            SpellStart( SvxSpellArea::Body );
             return true;
         }
         return false;
@@ -351,7 +351,7 @@ bool SvxSpellWrapper::SpellNext( )
     if ( bOtherCntnt )
     {
         bStartChk = false;
-        SpellStart( SVX_SPELL_BODY );
+        SpellStart( SvxSpellArea::Body );
         bGoOn = true;
     }
     else if ( bStartDone && bEndDone )
@@ -360,7 +360,7 @@ bool SvxSpellWrapper::SpellNext( )
         // Body area done, ask for special area
         if( !IsHyphen() && bIsSpellSpecial && HasOtherCnt() )
         {
-            SpellStart( SVX_SPELL_OTHER );
+            SpellStart( SvxSpellArea::Other );
             bOtherCntnt = bGoOn = true;
         }
         else if ( SpellMore() )  // check another document?
@@ -368,7 +368,7 @@ bool SvxSpellWrapper::SpellNext( )
             bOtherCntnt = false;
             bStartDone = !bReverse;
             bEndDone  = bReverse;
-            SpellStart( SVX_SPELL_BODY );
+            SpellStart( SvxSpellArea::Body );
             return true;
         }
     }
@@ -389,7 +389,7 @@ bool SvxSpellWrapper::SpellNext( )
         else
         {
             bStartChk = !bStartDone;
-            SpellStart( bStartChk ? SVX_SPELL_BODY_START : SVX_SPELL_BODY_END );
+            SpellStart( bStartChk ? SvxSpellArea::BodyStart : SvxSpellArea::BodyEnd );
             bGoOn = true;
         }
         WAIT_ON();
diff --git a/include/editeng/svxenum.hxx b/include/editeng/svxenum.hxx
index 9dd15ec..5c7c11ea 100644
--- a/include/editeng/svxenum.hxx
+++ b/include/editeng/svxenum.hxx
@@ -162,12 +162,12 @@ enum SvxDrawBezierEnum
     SVX_BEZIER_ELIMINATE_POINTS
 };
 
-enum SvxSpellArea
+enum class SvxSpellArea
 {
-    SVX_SPELL_BODY = 0,
-    SVX_SPELL_BODY_END,
-    SVX_SPELL_BODY_START,
-    SVX_SPELL_OTHER
+    Body = 0,
+    BodyEnd,
+    BodyStart,
+    Other
 };
 
 enum SvxFrameAnchor
diff --git a/sw/source/uibase/lingu/hhcwrp.cxx b/sw/source/uibase/lingu/hhcwrp.cxx
index a3b10db..8b48c37 100644
--- a/sw/source/uibase/lingu/hhcwrp.cxx
+++ b/sw/source/uibase/lingu/hhcwrp.cxx
@@ -600,11 +600,11 @@ void SwHHCWrapper::Convert()
     }
 
     if ( m_bIsOtherContent )
-        ConvStart_impl( m_pConvArgs, SVX_SPELL_OTHER );
+        ConvStart_impl( m_pConvArgs, SvxSpellArea::Other );
     else
     {
         m_bStartChk = false;
-        ConvStart_impl( m_pConvArgs, SVX_SPELL_BODY_END );
+        ConvStart_impl( m_pConvArgs, SvxSpellArea::BodyEnd );
     }
 
     ConvertDocument();
@@ -632,7 +632,7 @@ bool SwHHCWrapper::ConvNext_impl( )
     if ( m_bIsOtherContent )
     {
         m_bStartChk = false;
-        ConvStart_impl( m_pConvArgs, SVX_SPELL_BODY );
+        ConvStart_impl( m_pConvArgs, SvxSpellArea::Body );
         bGoOn = true;
     }
     else if ( m_bStartDone && m_bEndDone )
@@ -640,14 +640,14 @@ bool SwHHCWrapper::ConvNext_impl( )
         // body region done, ask about special region
         if( HasOtherCnt_impl() )
         {
-            ConvStart_impl( m_pConvArgs, SVX_SPELL_OTHER );
+            ConvStart_impl( m_pConvArgs, SvxSpellArea::Other );
             m_bIsOtherContent = bGoOn = true;
         }
     }
     else
     {
             m_bStartChk = !m_bStartDone;
-            ConvStart_impl( m_pConvArgs, m_bStartChk ? SVX_SPELL_BODY_START : SVX_SPELL_BODY_END );
+            ConvStart_impl( m_pConvArgs, m_bStartChk ? SvxSpellArea::BodyStart : SvxSpellArea::BodyEnd );
             bGoOn = true;
     }
     return bGoOn;
@@ -686,7 +686,7 @@ bool SwHHCWrapper::HasOtherCnt_impl()
 
 void SwHHCWrapper::ConvStart_impl( SwConversionArgs /* [out] */ *pConversionArgs, SvxSpellArea eArea )
 {
-    SetDrawObj( SVX_SPELL_OTHER == eArea );
+    SetDrawObj( SvxSpellArea::Other == eArea );
     m_pView->SpellStart( eArea, m_bStartDone, m_bEndDone, /* [out] */ pConversionArgs );
 }
 
diff --git a/sw/source/uibase/lingu/hyp.cxx b/sw/source/uibase/lingu/hyp.cxx
index dc2a33c..756c5c9 100644
--- a/sw/source/uibase/lingu/hyp.cxx
+++ b/sw/source/uibase/lingu/hyp.cxx
@@ -59,7 +59,7 @@ SwHyphWrapper::SwHyphWrapper( SwView* pVw,
 
 void SwHyphWrapper::SpellStart( SvxSpellArea eSpell )
 {
-    if( SVX_SPELL_OTHER == eSpell && nPageCount )
+    if( SvxSpellArea::Other == eSpell && nPageCount )
     {
         ::EndProgress( pView->GetDocShell() );
         nPageCount = 0;
diff --git a/sw/source/uibase/uiview/viewling.cxx b/sw/source/uibase/uiview/viewling.cxx
index 0514637..f900061 100644
--- a/sw/source/uibase/uiview/viewling.cxx
+++ b/sw/source/uibase/uiview/viewling.cxx
@@ -278,13 +278,13 @@ void SwView::SpellStart( SvxSpellArea eWhich,
     SwDocPositions eCurr  = DOCPOS_CURR;
     switch ( eWhich )
     {
-        case SVX_SPELL_BODY:
+        case SvxSpellArea::Body:
             if( bIsWrapReverse )
                 eCurr = DOCPOS_END;
             else
                 eCurr = DOCPOS_START;
             break;
-        case SVX_SPELL_BODY_END:
+        case SvxSpellArea::BodyEnd:
             if( bIsWrapReverse )
             {
                 if( bStartDone )
@@ -294,7 +294,7 @@ void SwView::SpellStart( SvxSpellArea eWhich,
             else if( bStartDone )
                 eCurr = DOCPOS_START;
             break;
-        case SVX_SPELL_BODY_START:
+        case SvxSpellArea::BodyStart:
             if( !bIsWrapReverse )
             {
                 if( bEndDone )
@@ -304,7 +304,7 @@ void SwView::SpellStart( SvxSpellArea eWhich,
             else if( bEndDone )
                 eCurr = DOCPOS_END;
             break;
-        case SVX_SPELL_OTHER:
+        case SvxSpellArea::Other:
             if( bIsWrapReverse )
             {
                 eStart = DOCPOS_OTHERSTART;
@@ -397,16 +397,16 @@ void SwView::HyphStart( SvxSpellArea eWhich )
 {
     switch ( eWhich )
     {
-        case SVX_SPELL_BODY:
+        case SvxSpellArea::Body:
             m_pWrtShell->HyphStart( DOCPOS_START, DOCPOS_END );
             break;
-        case SVX_SPELL_BODY_END:
+        case SvxSpellArea::BodyEnd:
             m_pWrtShell->HyphStart( DOCPOS_CURR, DOCPOS_END );
             break;
-        case SVX_SPELL_BODY_START:
+        case SvxSpellArea::BodyStart:
             m_pWrtShell->HyphStart( DOCPOS_START, DOCPOS_CURR );
             break;
-        case SVX_SPELL_OTHER:
+        case SvxSpellArea::Other:
             m_pWrtShell->HyphStart( DOCPOS_OTHERSTART, DOCPOS_OTHEREND );
             break;
         default:


More information about the Libreoffice-commits mailing list