fix usage of successful/successfully in comments

Change-Id: If280e1e944c877462cb84b579e19d4fd2ad1900b
Reviewed-on: https://gerrit.libreoffice.org/70813
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

11 files changed, 15 insertions, 15 deletions

diff --git a/framework/inc/dispatch/closedispatcher.hxx b/framework/inc/dispatch/closedispatcher.hxx
index 9aed7b5ae2cc..c6556259818f 100644
--- a/framework/inc/dispatch/closedispatcher.hxx
+++ b/framework/inc/dispatch/closedispatcher.hxx
@@ -176,7 +176,7 @@ class CloseDispatcher : public  ::cppu::WeakImplHelper<
                     We need it to implement the CLOSE_DOC semantic.
 
             @return [boolean]
-                    sal_True if closing was successfully.
+                    sal_True if closing was successful.
          */
         bool implts_prepareFrameForClosing(const css::uno::Reference< css::frame::XFrame >& xFrame,
                                            bool                                   bCloseAllOtherViewsToo,
@@ -194,7 +194,7 @@ class CloseDispatcher : public  ::cppu::WeakImplHelper<
                     frame is closed ....
 
             @return [bool]
-                    sal_True if closing was successfully.
+                    sal_True if closing was successful.
          */
         bool implts_closeFrame();
 
@@ -202,7 +202,7 @@ class CloseDispatcher : public  ::cppu::WeakImplHelper<
                     as new component of our m_xCloseFrame.
 
             @return [bool]
-                    sal_True if operation was successfully.
+                    sal_True if operation was successful.
          */
         bool implts_establishBackingMode();
 
diff --git a/framework/source/dispatch/closedispatcher.cxx b/framework/source/dispatch/closedispatcher.cxx
index d3c77c970fde..f6ac18d400ab 100644
--- a/framework/source/dispatch/closedispatcher.cxx
+++ b/framework/source/dispatch/closedispatcher.cxx
@@ -140,8 +140,8 @@ void SAL_CALL CloseDispatcher::dispatchWithNotification(const css::util::URL&
     // our asynchronous process was not finished yet.
     // We have to reject double calls. Otherwise we risk,
     // that we try to close an already closed resource ...
-    // And its no problem to do nothing then. The UI user will try it again, if
-    // non of these jobs was successfully.
+    // And it is no problem to do nothing then. The UI user will try it again, if
+    // non of these jobs was successful.
     if (m_xSelfHold.is())
     {
         aWriteLock.clear();
diff --git a/framework/source/dispatch/mailtodispatcher.cxx b/framework/source/dispatch/mailtodispatcher.cxx
index e2aabd3af857..696291e20f9f 100644
--- a/framework/source/dispatch/mailtodispatcher.cxx
+++ b/framework/source/dispatch/mailtodispatcher.cxx
@@ -170,7 +170,7 @@ void SAL_CALL MailToDispatcher::dispatchWithNotification( const css::util::URL&
 
     @return     <TRUE/> if dispatch could be started successfully
                 Note: Our internal used shell executor doesn't return any state value - so we must
-                believe that call was successfully.
+                believe that call was successful.
                 <FALSE/> if necessary resource couldn't be created or an exception was thrown.
 */
 bool MailToDispatcher::implts_dispatch( const css::util::URL& aURL )
diff --git a/framework/source/dispatch/servicehandler.cxx b/framework/source/dispatch/servicehandler.cxx
index cc4653dce95c..05b20432cd60 100644
--- a/framework/source/dispatch/servicehandler.cxx
+++ b/framework/source/dispatch/servicehandler.cxx
@@ -172,7 +172,7 @@ void SAL_CALL ServiceHandler::dispatchWithNotification( const css::util::URL&
 
     @return     <NULL/> if requested service couldn't be created successfully;
                 a valid reference otherwise. This return value can be used to indicate,
-                if dispatch was successfully or not.
+                if dispatch was successful.
 */
 css::uno::Reference< css::uno::XInterface > ServiceHandler::implts_dispatch( const css::util::URL& aURL )
 {
diff --git a/framework/source/inc/accelerators/keymapping.hxx b/framework/source/inc/accelerators/keymapping.hxx
index 9ce8cdcd8ed7..108cee42e56d 100644
--- a/framework/source/inc/accelerators/keymapping.hxx
+++ b/framework/source/inc/accelerators/keymapping.hxx
@@ -118,7 +118,7 @@ class KeyMapping
                     if this method returns sal_True!
 
             @return [boolean]
-                    sal_True if conversion was successfully.
+                    sal_True if conversion was successful.
           */
         bool impl_st_interpretIdentifierAsPureKeyCode(const OUString& sIdentifier,
                                                                 sal_uInt16&      rCode      );
diff --git a/framework/source/inc/accelerators/presethandler.hxx b/framework/source/inc/accelerators/presethandler.hxx
index b41593d41bee..55532b8a2a3f 100644
--- a/framework/source/inc/accelerators/presethandler.hxx
+++ b/framework/source/inc/accelerators/presethandler.hxx
@@ -297,7 +297,7 @@ class PresetHandler
             @param  bShare
                     force using of the share layer instead of the user layer.
 
-            @return An opened storage in case method was successfully - null otherwise.
+            @return An opened storage in case method was successful - null otherwise.
          */
         css::uno::Reference< css::embed::XStorage > impl_openPathIgnoringErrors(const OUString& sPath ,
                                                                                       sal_Int32        eMode ,
@@ -354,7 +354,7 @@ class PresetHandler
             @param  bAllowFallback
                     enable/disable fallback handling for locales
 
-            @return An opened storage in case method was successfully - null otherwise.
+            @return An opened storage in case method was successful - null otherwise.
          */
         css::uno::Reference< css::embed::XStorage > impl_openLocalizedPathIgnoringErrors(OUString&      sPath         ,
                                                                                          sal_Int32             eMode         ,
diff --git a/framework/source/inc/loadenv/loadenv.hxx b/framework/source/inc/loadenv/loadenv.hxx
index 597631488d82..4c8c148e41e9 100644
--- a/framework/source/inc/loadenv/loadenv.hxx
+++ b/framework/source/inc/loadenv/loadenv.hxx
@@ -293,7 +293,7 @@ public:
         @descr  The timeout parameter can be used to wait some times only
                 or forever. The return value indicates if the load request
                 was finished during the specified timeout period.
-                But it indicates not, if the load request was successfully or not!
+                But it indicates not, if the load request was successful or not!
 
         @param  nTimeout
                 specify a timeout in [ms].
diff --git a/framework/source/loadenv/loadenv.cxx b/framework/source/loadenv/loadenv.cxx
index 7855a5cd2072..e70f771eb053 100644
--- a/framework/source/loadenv/loadenv.cxx
+++ b/framework/source/loadenv/loadenv.cxx
@@ -758,7 +758,7 @@ void LoadEnv::impl_detectTypeAndFilter()
     // SAFE ->
     osl::ResettableMutexGuard aWriteLock(m_mutex);
 
-    // detection was successfully => update the descriptor member of this class
+    // detection was successful => update the descriptor member of this class
     m_lMediaDescriptor << lDescriptor;
     m_lMediaDescriptor[utl::MediaDescriptor::PROP_TYPENAME()] <<= sType;
     // Is there an already detected (may be preselected) filter?
diff --git a/framework/source/services/autorecovery.cxx b/framework/source/services/autorecovery.cxx
index 3f6098efef69..2f0922c02f09 100644
--- a/framework/source/services/autorecovery.cxx
+++ b/framework/source/services/autorecovery.cxx
@@ -737,7 +737,7 @@ private:
                 Note further: it patches the info struct
                 more than ones. E.g. the new temp URL is set
                 before the file is saved. And the old URL is removed
-                only if removing of the old file was successfully.
+                only if removing of the old file was successful.
                 If this method returns without an exception - everything
                 was OK. Otherwise the info struct can be analyzed to
                 get more information, e.g. when the problem occurs.
diff --git a/framework/source/services/desktop.cxx b/framework/source/services/desktop.cxx
index 964ca410945f..56d0b6886e3e 100644
--- a/framework/source/services/desktop.cxx
+++ b/framework/source/services/desktop.cxx
@@ -598,7 +598,7 @@ css::uno::Reference< css::frame::XFrame > SAL_CALL Desktop::getCurrentFrame()
     @param      "sTargetFrameName"  , name of target frame or special value like "_self", "_blank" ...
     @param      "nSearchFlags"      , optional arguments for frame search, if target isn't a special one
     @param      "lArguments"        , optional arguments for loading
-    @return     A valid component reference, if loading was successfully.
+    @return     A valid component reference, if loading was successful.
                 A null reference otherwise.
 
     @onerror    We return a null reference.
diff --git a/framework/source/services/frame.cxx b/framework/source/services/frame.cxx
index 2396ac9db8a0..ec44c24efd33 100644
--- a/framework/source/services/frame.cxx
+++ b/framework/source/services/frame.cxx
@@ -570,7 +570,7 @@ void XFrameImpl::initListeners()
     @param      "sTargetFrameName"  , name of target frame or special value like "_self", "_blank" ...
     @param      "nSearchFlags"      , optional arguments for frame search, if target is not a special one
     @param      "lArguments"        , optional arguments for loading
-    @return     A valid component reference, if loading was successfully.
+    @return     A valid component reference, if loading was successful.
                 A null reference otherwise.
 
     @onerror    We return a null reference.