[Libreoffice-commits] core.git: sfx2/source shell/source

Stephan Bergmann (via logerrit) logerrit at kemper.freedesktop.org
Tue Oct 1 11:47:18 UTC 2019


 sfx2/source/appl/shutdowniconw32.cxx             |    2 +-
 shell/source/win32/simplemail/smplmailclient.cxx |    3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

New commits:
commit 2a662212982d5baa3b69d34b4074d93d39287898
Author:     Stephan Bergmann <sbergman at redhat.com>
AuthorDate: Tue Oct 1 10:20:16 2019 +0200
Commit:     Stephan Bergmann <sbergman at redhat.com>
CommitDate: Tue Oct 1 13:46:31 2019 +0200

    loplugin:sequenceloop (clang-cl)
    
    Change-Id: Ie55967043fd6584c8b9a0ad66fad118c70bda8f0
    Reviewed-on: https://gerrit.libreoffice.org/79927
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sbergman at redhat.com>

diff --git a/sfx2/source/appl/shutdowniconw32.cxx b/sfx2/source/appl/shutdowniconw32.cxx
index b6fd1741a360..f5e836dfbc5a 100644
--- a/sfx2/source/appl/shutdowniconw32.cxx
+++ b/sfx2/source/appl/shutdowniconw32.cxx
@@ -149,7 +149,7 @@ static HMENU createSystrayMenu( )
     // collect the URLs of the entries in the File/New menu
     ::std::set< OUString > aFileNewAppsAvailable;
     SvtDynamicMenuOptions aOpt;
-    Sequence < Sequence < PropertyValue > > aNewMenu = aOpt.GetMenu( EDynamicMenuType::NewMenu );
+    Sequence < Sequence < PropertyValue > > const aNewMenu = aOpt.GetMenu( EDynamicMenuType::NewMenu );
     const OUString sURLKey( "URL"  );
 
     for ( auto const & newMenuProp : aNewMenu )
diff --git a/shell/source/win32/simplemail/smplmailclient.cxx b/shell/source/win32/simplemail/smplmailclient.cxx
index 7b225d16975b..45331098b835 100644
--- a/shell/source/win32/simplemail/smplmailclient.cxx
+++ b/shell/source/win32/simplemail/smplmailclient.cxx
@@ -303,7 +303,8 @@ void CSmplMailClient::assembleCommandLine(
         rCommandArgs.push_back(subject);
     }
 
-    for (const auto& attachment : xSimpleMailMessage->getAttachement())
+    auto const attachments = xSimpleMailMessage->getAttachement();
+    for (const auto& attachment : attachments)
     {
         OUString sDisplayName;
         OUString sTempFileURL(CopyAttachment(attachment, sDisplayName));


More information about the Libreoffice-commits mailing list