diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-01-27 16:09:54 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-01-28 09:22:55 +0000 |
| commit | f1d83ac45f08270f7f2dd7128056effd0251dc5e (patch) | |
| tree | 55d924eaa7f55627039d44458d869ef65130fdf8 /ucb/source/ucp/cmis | |
| parent | 53d3755972bfd3bd2cd650edf91f1483038028c8 (diff) | |
loplugin:stringconstant check for unnecessary OUString constructor..
..calls when creating exceptions
Change-Id: I3bc58a5aa4dc6f0508ecb88b3a843b96b8c7ebfe
Reviewed-on: https://gerrit.libreoffice.org/33617
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'ucb/source/ucp/cmis')
| -rw-r--r-- | ucb/source/ucp/cmis/cmis_content.cxx | 12 | ||||
| -rw-r--r-- | ucb/source/ucp/cmis/cmis_repo_content.cxx | 2 |
2 files changed, 7 insertions, 7 deletions
diff --git a/ucb/source/ucp/cmis/cmis_content.cxx b/ucb/source/ucp/cmis/cmis_content.cxx index 7d0ab609edce..5296e79fa4b8 100644 --- a/ucb/source/ucp/cmis/cmis_content.cxx +++ b/ucb/source/ucp/cmis/cmis_content.cxx @@ -595,7 +595,7 @@ namespace cmis uno::Any Content::getBadArgExcept() { return uno::makeAny( lang::IllegalArgumentException( - OUString("Wrong argument type!"), + "Wrong argument type!", static_cast< cppu::OWeakObject * >( this ), -1) ); } @@ -1272,7 +1272,7 @@ namespace cmis ucbhelper::cancelCommandExecution( uno::makeAny( ucb::InteractiveBadTransferURLException( - OUString("Unsupported URL scheme!"), + "Unsupported URL scheme!", static_cast< cppu::OWeakObject * >( this ) ) ), xEnv ); } @@ -1479,7 +1479,7 @@ namespace cmis rValue.Name == "Size" || rValue.Name == "CreatableContentsInfo" ) { - lang::IllegalAccessException e ( OUString("Property is read-only!"), + lang::IllegalAccessException e ( "Property is read-only!", static_cast< cppu::OWeakObject* >( this ) ); aRet[ n ] <<= e; } @@ -1489,7 +1489,7 @@ namespace cmis if (!( rValue.Value >>= aNewTitle )) { aRet[ n ] <<= beans::IllegalTypeException - ( OUString("Property value has wrong type!"), + ( "Property value has wrong type!", static_cast< cppu::OWeakObject * >( this ) ); continue; } @@ -1497,7 +1497,7 @@ namespace cmis if ( aNewTitle.getLength() <= 0 ) { aRet[ n ] <<= lang::IllegalArgumentException - ( OUString("Empty title not allowed!"), + ( "Empty title not allowed!", static_cast< cppu::OWeakObject * >( this ), -1 ); continue; @@ -1509,7 +1509,7 @@ namespace cmis else { SAL_INFO( "ucb.ucp.cmis", "Couldn't set property: " << rValue.Name ); - lang::IllegalAccessException e ( OUString("Property is read-only!"), + lang::IllegalAccessException e ( "Property is read-only!", static_cast< cppu::OWeakObject* >( this ) ); aRet[ n ] <<= e; } diff --git a/ucb/source/ucp/cmis/cmis_repo_content.cxx b/ucb/source/ucp/cmis/cmis_repo_content.cxx index a1a4bd0bedcd..a008c9b92dff 100644 --- a/ucb/source/ucp/cmis/cmis_repo_content.cxx +++ b/ucb/source/ucp/cmis/cmis_repo_content.cxx @@ -72,7 +72,7 @@ namespace cmis uno::Any RepoContent::getBadArgExcept() { return uno::makeAny( lang::IllegalArgumentException( - OUString("Wrong argument type!"), + "Wrong argument type!", static_cast< cppu::OWeakObject * >( this ), -1) ); } |