diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-10-28 10:35:49 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-10-28 12:07:54 +0100 |
commit | bc539bd3c04964471af6b5ca54264215c8995696 (patch) | |
tree | c37af5b9fb3dc0b8677fc8dd535fb8e93ad6039d /sc/source/ui/unoobj | |
parent | 9f78a4174e5099ad3af65a23e158a51c1afca54d (diff) |
loplugin:stringadd improve detection
if one side of the expression is a compile-time-constant, we don't need
to worry about side-effects on the other side
Change-Id: Iee71ea51b327ef244bf39f128f921ac325d74e2b
Reviewed-on: https://gerrit.libreoffice.org/81589
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sc/source/ui/unoobj')
-rw-r--r-- | sc/source/ui/unoobj/cellvaluebinding.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sc/source/ui/unoobj/cellvaluebinding.cxx b/sc/source/ui/unoobj/cellvaluebinding.cxx index 8063ecf4b54e..a5d43ab012e2 100644 --- a/sc/source/ui/unoobj/cellvaluebinding.cxx +++ b/sc/source/ui/unoobj/cellvaluebinding.cxx @@ -405,9 +405,9 @@ namespace calc OCellValueBinding* pNonConstThis = const_cast< OCellValueBinding* >( this ); if ( !pNonConstThis->supportsType( _rType ) ) { - OUString sMessage( "The given type (" ); - sMessage += _rType.getTypeName(); - sMessage += ") is not supported by this binding."; + OUString sMessage = "The given type (" + + _rType.getTypeName() + + ") is not supported by this binding."; // TODO: localize this error message throw IncompatibleTypesException( sMessage, *pNonConstThis ); |