diff --git a/ucb/source/ucp/webdav-curl/DAVTypes.cxx b/ucb/source/ucp/webdav-curl/DAVTypes.cxx
index 58cb126dde62..019f6e582c67 100644
--- a/ucb/source/ucp/webdav-curl/DAVTypes.cxx
+++ b/ucb/source/ucp/webdav-curl/DAVTypes.cxx
@@ -123,7 +123,6 @@ bool DAVOptionsCache::getDAVOptions( const OUString & rURL, DAVOptions & rDAVOpt
             // if stale, remove from cache, do not restore
             m_aTheCache.erase( it );
             return false;
-            // return false instead
         }
         rDAVOptions = (*it).second;
         return true;
diff --git a/unotools/source/accessibility/accessiblerelationsethelper.cxx b/unotools/source/accessibility/accessiblerelationsethelper.cxx
index 72bc491b979b..9dd298f00acd 100644
--- a/unotools/source/accessibility/accessiblerelationsethelper.cxx
+++ b/unotools/source/accessibility/accessiblerelationsethelper.cxx
@@ -103,7 +103,7 @@ sal_Int32 SAL_CALL
     /** Tests whether the relation set contains a relation matching the
         specified key.
 
-        @param aRelationType
+        @param eRelationType
             The type of relation to look for in this set of relations.  This
             has to be one of the constants of
             <type>AccessibleRelationType</type>.
diff --git a/unotools/source/config/historyoptions.cxx b/unotools/source/config/historyoptions.cxx
index c4fd22bc7177..4b856def7ddf 100644
--- a/unotools/source/config/historyoptions.cxx
+++ b/unotools/source/config/historyoptions.cxx
@@ -402,7 +402,7 @@ void TogglePinItem(EHistoryType eHistory, const OUString& sURL)
             uno::Reference<container::XNameContainer> xOrderList;
             xListAccess->getByName(s_sOrderList) >>= xOrderList;
 
-            // Shift item to the beginning of the document list if is not pinned now
+            // Shift item to the beginning of the document list if it is not pinned now
             if (bIsItemPinned)
                 MoveItemToUnpinned(xCfg, xOrderList, xItemList, sURL);
             else