Scope breaks; and add number comments to SwFieldIds

SwFieldIds is fixed, so add number comments to easier find the ID
in the long list.

Change-Id: I2b9fc00bd0ddc15fc327324c2895810d61e2870e
Reviewed-on: https://gerrit.libreoffice.org/60795
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>

sw/inc/fldbas.hxx

@@ -43,7 +43,7 @@ enum class SwFieldIds : sal_uInt16 {
     Author,
     Chapter,
     DocStat,
-    GetExp,
+    GetExp, // 10
     SetExp,
     GetRef,
     HiddenText,
@@ -53,7 +53,7 @@ enum class SwFieldIds : sal_uInt16 {
     Reg,
     VarReg,
     SetRef,
-    Input,
+    Input, // 20
     Macro,
     Dde,
     Table,
@@ -63,7 +63,7 @@ enum class SwFieldIds : sal_uInt16 {
     DbNextSet,
     DbNumSet,
     DbSetNumber,
-    ExtUser,
+    ExtUser, // 30
     RefPageSet,
     RefPageGet,
     Internet,
@@ -73,7 +73,7 @@ enum class SwFieldIds : sal_uInt16 {
     TableOfAuthorities,
     CombinedChars,
     Dropdown,
-    ParagraphSignature,
+    ParagraphSignature, // 40
     LAST = ParagraphSignature,
 
     Unknown = USHRT_MAX, // used as default value in some method calls

sw/source/uibase/docvw/edtwin2.cxx

@@ -160,8 +160,8 @@ void SwEditWin::RequestHelp(const HelpEvent &rEvt)
                 sText = OStringToOUString(OString::number(
                             static_cast<const SwTableBoxValue*>(aContentAtPos.aFnd.pAttr)->GetValue()),
                             osl_getThreadTextEncoding());
+                break;
             }
-            break;
             case IsAttrAtPos::CurrAttrs:
                 sText = aContentAtPos.sStr;
                 break;
@@ -241,8 +241,8 @@ void SwEditWin::RequestHelp(const HelpEvent &rEvt)
                 aModStr = aModStr.replaceFirst(aCode.GetName(), "");
                 aModStr = aModStr.replaceAll("+", "");
                 sText = SwResId(STR_SMARTTAG_CLICK).replaceAll("%s", aModStr);
+                break;
             }
-            break;
 
             case IsAttrAtPos::Ftn:
                 if( aContentAtPos.pFndTextAttr && aContentAtPos.aFnd.pAttr )
@@ -264,8 +264,8 @@ void SwEditWin::RequestHelp(const HelpEvent &rEvt)
                 const bool bShowInlineTooltips = rSh.GetViewOptions()->IsShowInlineTooltips();
                 if ( bShowTrackChanges && bShowInlineTooltips )
                      sText = lcl_GetRedlineHelp(*aContentAtPos.aFnd.pRedl, bBalloon);
+                break;
             }
-            break;
 
             case IsAttrAtPos::ToxMark:
                 sText = aContentAtPos.sStr;
@@ -280,6 +280,7 @@ void SwEditWin::RequestHelp(const HelpEvent &rEvt)
                     }
                 }
                 break;
+
             case IsAttrAtPos::RefMark:
                 if(aContentAtPos.aFnd.pAttr)
                 {
@@ -357,8 +358,9 @@ void SwEditWin::RequestHelp(const HelpEvent &rEvt)
                                     sText = static_cast<const SwGetRefField*>(pField)->GetSetRefName();
                                 }
                             }
+                            break;
                         }
-                        break;
+
                         default: break;
                         }
                     }