fpicker/source/office/RemoteFilesDialog.cxx | 22 +++++++++------------- 1 file changed, 9 insertions(+), 13 deletions(-)
New commits: commit 48fc2fee15a7d9b43da430598e3acfa18e739250 Author: Matteo Casalin <matteo.casa...@yahoo.com> AuthorDate: Mon Mar 4 17:47:20 2019 +0100 Commit: Matteo Casalin <matteo.casa...@yahoo.com> CommitDate: Thu Mar 14 22:43:08 2019 +0100 getToken() ==> copy() since separator positions are known Change-Id: Ib13f279fd764538deb77b3ee871e58baf8676be2 Reviewed-on: https://gerrit.libreoffice.org/69242 Tested-by: Jenkins Reviewed-by: Matteo Casalin <matteo.casa...@yahoo.com> diff --git a/fpicker/source/office/RemoteFilesDialog.cxx b/fpicker/source/office/RemoteFilesDialog.cxx index 850490ae9041..f0491762dde8 100644 --- a/fpicker/source/office/RemoteFilesDialog.cxx +++ b/fpicker/source/office/RemoteFilesDialog.cxx @@ -462,19 +462,15 @@ void RemoteFilesDialog::InitSize() OUString sCfgStr; if( aUserData >>= sCfgStr ) { - int nPos = sCfgStr.indexOf( "|" ); - if( nPos != -1 ) - { - nPos = sCfgStr.indexOf( "|", nPos + 1 ); - if( nPos != -1 ) - { - sal_Int32 nIdx = 0; - m_nWidth = sCfgStr.getToken( 0, '|', nIdx ).toInt32(); - m_nHeight = sCfgStr.getToken( 0, '|', nIdx ).toInt32(); - - m_pFileView->SetConfigString( sCfgStr.copy( nPos + 1) ); - } - } + sal_Int32 nPos1{ sCfgStr.indexOf('|') }; + if (nPos1<0) + return; + sal_Int32 nPos2{ sCfgStr.indexOf('|', nPos1+1 ) }; + if (nPos2<0) + return; + m_nWidth = sCfgStr.copy(0, nPos1++).toInt32(); + m_nHeight = sCfgStr.copy(nPos1, nPos2-nPos1).toInt32(); + m_pFileView->SetConfigString( sCfgStr.copy(nPos2+1) ); } } else _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits