diff options
| author | Eike Rathke <erack@redhat.com> | 2016-10-27 00:28:44 +0200 |
|---|---|---|
| committer | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2016-10-27 03:29:55 +0000 |
| commit | fac78d1d350184319d91d4f7d15bb3f22dccf0da (patch) | |
| tree | b288d9d3fc899b21f1179d1fe5fea8700fad8efe /sc | |
| parent | e2dc12fa1803d5e3d6a06b2533d04c1bc54c8113 (diff) | |
trust the size ... tdf#79442 follow-up
... and don't unnecessarily loop over the whole range.
Actually a max/min typo.
Change-Id: I3dbda3a7388baa337c3922ad992fd3946074c3b7
(cherry picked from commit 329ecb67808767d677383e5fd6089feaab4cd9e4)
Reviewed-on: https://gerrit.libreoffice.org/30312
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/ui/docshell/externalrefmgr.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sc/source/ui/docshell/externalrefmgr.cxx b/sc/source/ui/docshell/externalrefmgr.cxx index 59863f68efbd..546f10226306 100644 --- a/sc/source/ui/docshell/externalrefmgr.cxx +++ b/sc/source/ui/docshell/externalrefmgr.cxx @@ -2652,7 +2652,7 @@ void ScExternalRefManager::addFilesToLinkManager() SAL_WARN_IF( maSrcFiles.size() >= SAL_MAX_UINT16, "sc.ui", "ScExternalRefManager::addFilesToLinkManager: files overflow"); - const sal_uInt16 nSize = static_cast<sal_uInt16>( std::max<size_t>( maSrcFiles.size(), SAL_MAX_UINT16)); + const sal_uInt16 nSize = static_cast<sal_uInt16>( std::min<size_t>( maSrcFiles.size(), SAL_MAX_UINT16)); for (sal_uInt16 nFileId = 0; nFileId < nSize; ++nFileId) maybeLinkExternalFile( nFileId); } |