Fix typos in code and comments

Change-Id: I48caad6ed035a1e46e434a685fc40376e0276fe5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/210576
Tested-by: Julien Nabet <serval2412@yahoo.fr>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

sw/source/core/text/itrform2.cxx

@@ -83,7 +83,7 @@ namespace {
     //! Determine if we need to build hidden portions
     bool lcl_BuildHiddenPortion(const SwTextSizeInfo& rInf, TextFrameIndex &rPos);
 
-    // Check whether the two font has the same border
+    // Check whether the two fonts have the same border
     bool lcl_HasSameBorder(const SwFont& rFirst, const SwFont& rSecond);
 }
 
@@ -1435,7 +1435,7 @@ SwTextPortion *SwTextFormatter::WhichTextPor( SwTextFormatInfo &rInf ) const
                     {
                         pPor = new SwFieldFormDropDownPortion(pBM, sw::mark::ExpandFieldmark(pBM));
                     }
-                    /* we need to check for ODF_FORMTEXT for scenario having FormFields inside FORMTEXT.
+                    /* We need to check for ODF_FORMTEXT for scenarios having FormFields inside FORMTEXT.
                      * Otherwise file will crash on open.
                      */
                     else if (pBM->GetFieldname( ) == ODF_FORMTEXT)

sw/source/core/text/portxt.hxx

@@ -26,7 +26,7 @@ class SwTextGuess;
 class SwTextPortion : public SwLinePortion
 {
     void BreakCut( SwTextFormatInfo &rInf, const SwTextGuess &rGuess );
-    // is this hyphenation is good enough for a compound word, i.e.
+    // is this hyphenation good enough for a compound word, i.e.
     // it splits a compound word between compound constituents or
     // it hyphenates the compound word just before or inside its suffix
     static bool IsCompoundSplit( SwTextFormatInfo &rInf, const SwTextGuess &rGuess,

sw/source/core/txtnode/modeltoviewhelper.cxx

@@ -57,10 +57,6 @@ public:
     }
 };
 
-}
-
-namespace {
-
 struct block
 {
     sal_Int32 m_nStart;

sw/source/core/txtnode/swfont.cxx

@@ -1523,7 +1523,7 @@ void SwDrawTextInfo::Shift( Degree10 nDir )
 SwUnderlineFont::SwUnderlineFont(SwFont& rFnt, TextFrameIndex const nEnd, const Point& rPoint)
         : m_aPos( rPoint ), m_nEnd( nEnd ), m_pFont( &rFnt )
 {
-};
+}
 
 SwUnderlineFont::~SwUnderlineFont()
 {

sw/source/core/view/viewsh.cxx

@@ -1700,7 +1700,7 @@ bool SwViewShell::SmoothScroll( tools::Long lXDiff, tools::Long lYDiff, const to
                             rTargetDevice.EnableMapMode(bMapModeWasEnabledDest);
                             pVout->EnableMapMode(bMapModeWasEnabledSource);
 
-                            // end paint on logoc base
+                            // end paint on logic base
                             DLPostPaint2(true);
                     }
                     else

sw/source/filter/html/css1atr.cxx

@@ -3611,7 +3611,7 @@ SwAttrFnTab const aCSS1AttrFnTab = {
 /* RES_GRFATR_GAMMA */              nullptr,
 /* RES_GRFATR_INVERT */             nullptr,
 /* RES_GRFATR_TRANSPARENCY */       nullptr,
-/* RES_GRFATR_DRWAMODE */           nullptr,
+/* RES_GRFATR_DRAWMODE */           nullptr,
 /* RES_GRFATR_DUMMY3 */             nullptr,
 /* RES_GRFATR_VISIBLE */            nullptr,
 /* RES_GRFATR_DUMMY5 */             nullptr,

sw/source/filter/html/htmlatr.cxx

@@ -3440,7 +3440,7 @@ const SwAttrFnTab aHTMLAttrFnTab = {
 /* RES_GRFATR_GAMMA */              nullptr,
 /* RES_GRFATR_INVERT */             nullptr,
 /* RES_GRFATR_TRANSPARENCY */       nullptr,
-/* RES_GRFATR_DRWAMODE */           nullptr,
+/* RES_GRFATR_DRAWMODE */           nullptr,
 /* RES_GRFATR_DUMMY3 */             nullptr,
 /* RES_GRFATR_VISIBLE */            nullptr,
 /* RES_GRFATR_DUMMY5 */             nullptr,

sw/source/filter/ww8/ww8par.cxx

@@ -184,7 +184,7 @@ static SwMacroInfo* GetMacroInfo( SdrObject* pObj )
     }
 
     return nullptr;
-};
+}
 
 static void lclGetAbsPath(OUString& rPath, sal_uInt16 nLevel, SwDocShell const * pDocShell)
 {

sw/source/ui/table/instable.cxx

@@ -174,7 +174,7 @@ IMPL_LINK(SwInsTableDlg, ModifyName, weld::TextWidget&, rEdit, void)
 }
 
 // We use weld::Entry's "changed" notification here, not weld::SpinButton's "value_changed", because
-// the latter only fires after the control looses focus; so the notification would not fire during
+// the latter only fires after the control loses focus; so the notification would not fire during
 // typing a big number, so that user typing it and immediately clicking "Insert" would not see the
 // warning.
 // Since the notification is called in weld::Entry context, we can only rely on what's available for

sw/source/ui/table/tabledlg.cxx

@@ -164,8 +164,8 @@ IMPL_LINK( SwFormatTablePage, RelWidthClickHdl, weld::Toggleable&, rBtn, void )
         m_xWidthMF->SetRefValue(m_pTableData->GetSpace());
         m_xLeftMF->SetRefValue(m_pTableData->GetSpace());
         m_xRightMF->SetRefValue(m_pTableData->GetSpace());
-        m_xLeftMF->SetMetricFieldMin(0); //will be overwritten by the Percentfield
-        m_xRightMF->SetMetricFieldMin(0); //ditto
+        m_xLeftMF->SetMetricFieldMin(0); // will be overwritten by the Percentfield
+        m_xRightMF->SetMetricFieldMin(0); // ditto
         m_xLeftMF->SetMetricFieldMax(99);
         m_xRightMF->SetMetricFieldMax(99);
         m_xLeftMF->set_value(m_xLeftMF->NormalizePercent(nLeft ), FieldUnit::TWIP );

sw/source/uibase/app/docsh.cxx

@@ -1278,7 +1278,7 @@ void SwDocShell::RemoveOLEObjects()
 // size change takes place later, while calculating the layout in the
 // idle handler. If this document is saved now, it is saved with invalid
 // sizes. For this reason, the layout has to be calculated before a document is
-// saved, but of course only id there are OLE objects with bOLESizeInvalid set.
+// saved, but of course only if there are OLE objects with bOLESizeInvalid set.
 void SwDocShell::CalcLayoutForOLEObjects()
 {
     if (!m_pWrtShell)

sw/source/uibase/uno/unotxvw.cxx

@@ -1435,7 +1435,7 @@ OUString SwXTextViewCursor::getString()
             //! since setString for SEL_TABLE_TEXT (with possible
             //! multi selection of cells) would not work properly we
             //! will ignore this case for both
-            //! functions (setString AND getString) because of symmetrie.
+            //! functions (setString AND getString) because of symmetry.
 
             case ShellMode::ListText       :
             case ShellMode::TableListText:
@@ -1468,7 +1468,7 @@ void SwXTextViewCursor::setString(const OUString& aString)
         //! since setString for SEL_TABLE_TEXT (with possible
         //! multi selection of cells) would not work properly we
         //! will ignore this case for both
-        //! functions (setString AND getString) because of symmetrie.
+        //! functions (setString AND getString) because of symmetry.
 
         case ShellMode::ListText       :
         case ShellMode::TableListText :

sw/source/uibase/utlui/content.cxx

@@ -163,7 +163,7 @@ namespace {
 
 /*
     Symbolic name representations of numeric values used for the Outline Content Visibility popup
-    menu item ids. The numbers are chosen arbitrarily to not over overlap other menu item ids.
+    menu item ids. The numbers are chosen arbitrarily to not overlap other menu item ids.
     see: SwContentTree::ExecuteContextMenuAction, navigatorcontextmenu.ui
 
     1512 toggle outline content visibility of the selected outline entry

sw/source/writerfilter/dmapper/DomainMapper_Impl.cxx

@@ -10245,7 +10245,7 @@ void DomainMapper_Impl::deferCharacterProperty(sal_Int32 id, const css::uno::Any
 
 void DomainMapper_Impl::processDeferredCharacterProperties(bool bCharContext)
 {
-    // Actually process in DomainMapper, so that it's the same source file like normal processing.
+    // Actually process in DomainMapper, so that it's the same source file as normal processing.
     if (!m_StreamStateStack.top().deferredCharacterProperties.empty())
     {
         m_rDMapper.processDeferredCharacterProperties(m_StreamStateStack.top().deferredCharacterProperties, bCharContext);

sw/source/writerfilter/dmapper/SettingsTable.hxx

@@ -46,7 +46,7 @@ public:
     SettingsTable(const DomainMapper& rDomainMapper);
     virtual ~SettingsTable() override;
 
-    //returns default TabStop in 1/100th mm
+    /// Returns default TabStop in 1/100th mm.
     int GetDefaultTabStop() const;
 
     /// Automatically update styles from document template?

sw/source/writerfilter/inc/ooxml/OOXMLDocument.hxx

@@ -250,7 +250,7 @@ public:
        Resolves a comment to a stream handler.
 
        @param rStream       stream handler to resolve to
-       @param rComment      id of the comment to resolve
+       @param nId           id of the comment to resolve
      */
     void resolveComment(Stream & rStream, const sal_Int32 nId);
 

sw/source/writerfilter/rtftok/rtfsdrimport.cxx

@@ -245,7 +245,7 @@ void RTFSdrImport::applyProperty(uno::Reference<drawing::XShape> const& xShape,
     else if (aKey == u"rotation")
     {
         // See DffPropertyReader::Fix16ToAngle(): in RTF, positive rotation angles are clockwise, we have them as counter-clockwise.
-        // Additionally, RTF type is 0..360*2^16, our is 0..360*100.
+        // Additionally, RTF type is 0..360*2^16, ours is 0..360*100.
         sal_Int32 nRotation = o3tl::toInt32(aValue) * 100 / RTF_MULTIPLIER;
         uno::Reference<lang::XServiceInfo> xServiceInfo(xShape, uno::UNO_QUERY);
         if (!xServiceInfo->supportsService(u"com.sun.star.text.TextFrame"_ustr))

test/source/unoapi_test.cxx

@@ -188,7 +188,7 @@ void UnoApiTest::validate(TestFilter eFilter)
     }
 
 #if !defined _WIN32
-    // For now, this is only needed by some Linux ASan builds, so keep it simply and disable it on
+    // For now, this is only needed by some Linux ASan builds, so keep it simple and disable it on
     // Windows (which doesn't support the relevant shell syntax for (un-)setting environment
     // variables).
     OUString env;

toolkit/qa/complex/toolkit/accessibility/_XAccessibleContext.java

@@ -58,7 +58,7 @@ public class _XAccessibleContext {
 
     /**
      * Calls the method and stores the number of children. <p>
-     * Has <b> OK </b> status if non-negative number returned.
+     * Has <b> OK </b> status if a non-negative number is returned.
      */
     public boolean _getAccessibleChildCount() {
         childCount = oObj.getAccessibleChildCount();
@@ -69,8 +69,8 @@ public class _XAccessibleContext {
     /**
      * Tries to get every child and checks its parent. <p>
      *
-     * Has <b> OK </b> status if parent of every child
-     * and the tested component are the same objects.
+     * Has <b> OK </b> status if the parent of every child
+     * and the tested component are the same object.
      *
      * The following method tests are to be completed successfully before :
      * <ul>
@@ -141,7 +141,7 @@ public class _XAccessibleContext {
      * it with tested component.<p>
      *
      * Has <b> OK </b> status if the parent's child and the tested
-     * component are the same objects.
+     * component are the same object.
      *
      * The following method tests are to be completed successfully before :
      * <ul>
@@ -172,7 +172,7 @@ public class _XAccessibleContext {
     /**
      * Get the accessible role of component. <p>
      *
-     * Has <b> OK </b> status if non-negative number returned.
+     * Has <b> OK </b> status if a non-negative number is returned.
      */
     public boolean _getAccessibleRole() {
         short role = oObj.getAccessibleRole();
@@ -183,7 +183,7 @@ public class _XAccessibleContext {
     /**
      * Get the accessible name of the component. <p>
      *
-     * Has <b> OK </b> status if the name has non-zero length.
+     * Has <b> OK </b> status if the name is not null.
      */
     public boolean _getAccessibleName() {
         String name = oObj.getAccessibleName();
@@ -194,7 +194,7 @@ public class _XAccessibleContext {
     /**
      * Get the accessible description of the component. <p>
      *
-     * Has <b> OK </b> status if the description has non-zero length.
+     * Has <b> OK </b> status if the description is not null.
      */
     public boolean _getAccessibleDescription() {
         String descr = oObj.getAccessibleDescription();

toolkit/source/controls/unocontrol.cxx

@@ -156,7 +156,7 @@ Reference< XVclWindowPeer >    UnoControl::ImplGetCompatiblePeer()
 
     if ( !xCompatiblePeer.is() )
     {
-        // Create the pair as invisible
+        // Create the peer as invisible
         bool bVis = maComponentInfos.bVisible;
         if( bVis )
             maComponentInfos.bVisible = false;