[Libreoffice-commits] core.git: linguistic/source

Stephan Bergmann (via logerrit) logerrit at kemper.freedesktop.org
Tue Aug 4 18:48:51 UTC 2020


 linguistic/source/dlistimp.cxx |    8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

New commits:
commit 5a28a2ac3f692975306025e9e35e95f4c9ddcd38
Author:     Stephan Bergmann <sbergman at redhat.com>
AuthorDate: Tue Aug 4 14:30:04 2020 +0200
Commit:     Stephan Bergmann <sbergman at redhat.com>
CommitDate: Tue Aug 4 20:48:15 2020 +0200

    loplugin:referencecasting (clang-cl)
    
    ("the source reference is already a subtype of the destination reference")
    
    Change-Id: Ifba02593d77057eaa356ec7c3c76de64ba65cf09
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100089
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sbergman at redhat.com>

diff --git a/linguistic/source/dlistimp.cxx b/linguistic/source/dlistimp.cxx
index 31f0a20c4230..03c576a820f4 100644
--- a/linguistic/source/dlistimp.cxx
+++ b/linguistic/source/dlistimp.cxx
@@ -530,10 +530,8 @@ void SAL_CALL
         size_t nCount = rDicList.size();
         for (size_t i = 0;  i < nCount;  i++)
         {
-            uno::Reference< XDictionary > xDic( rDicList[i], UNO_QUERY );
-
             // save (modified) dictionaries
-            uno::Reference< frame::XStorable >  xStor( xDic , UNO_QUERY );
+            uno::Reference< frame::XStorable >  xStor( rDicList[i] , UNO_QUERY );
             if (xStor.is())
             {
                 try
@@ -548,8 +546,8 @@ void SAL_CALL
 
             // release references to (members of) this object hold by
             // dictionaries
-            if (xDic.is())
-                xDic->removeDictionaryEventListener( mxDicEvtLstnrHelper.get() );
+            if (rDicList[i].is())
+                rDicList[i]->removeDictionaryEventListener( mxDicEvtLstnrHelper.get() );
         }
     }
     mxDicEvtLstnrHelper.clear();


More information about the Libreoffice-commits mailing list