diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2021-07-09 13:04:19 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2021-07-09 15:07:05 +0200 |
| commit | f020784e14a55c82418e4f231855040177ac9f82 (patch) | |
| tree | a2d5c1d2b5778e564b9ee529de0453c73907ce70 /ucb | |
| parent | ac7bba3cc1e13824732f5ab69af602848e4ba227 (diff) | |
Make loplugin:stringadd slightly more aggressive
...by assuming that all const member functions are side-effect free. (This
presumably means that some of the special cases in
StringAdd::isSideEffectFree are obsoleted by this more general case, but any
such removal is postponed to later clean-up.)
(Came across this when idly wondering why
8b7f948d9d79393bc6c1b11d239706666fd5d7de "sc, VmlFormControlExporter: avoid
OStringBuffer style" had not been found by the plugin before.)
Change-Id: I6bca10df53885b14a590543aabd61f23b3748572
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118675
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'ucb')
| -rw-r--r-- | ucb/source/ucp/hierarchy/hierarchycontent.cxx | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/ucb/source/ucp/hierarchy/hierarchycontent.cxx b/ucb/source/ucp/hierarchy/hierarchycontent.cxx index 76391b4bfc51..c92264fc4c16 100644 --- a/ucb/source/ucp/hierarchy/hierarchycontent.cxx +++ b/ucb/source/ucp/hierarchy/hierarchycontent.cxx @@ -1365,9 +1365,7 @@ void HierarchyContent::insert( sal_Int32 nNameClashResolve, } else { - OUString aNewTitle( m_aProps.getTitle() ); - aNewTitle += "_" + - OUString::number( nTry ); + OUString aNewTitle( m_aProps.getTitle() + "_" + OUString::number( nTry ) ); m_aProps.setTitle( aNewTitle ); } } |