diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-08-04 08:45:36 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-08-04 11:21:36 +0200 |
| commit | cda88ec7a43162df37098c4525c33451266a1575 (patch) | |
| tree | 4f80310ab0ef1210b7dcd4d769b70c59fbc09f1e /desktop/source/deployment | |
| parent | f45ff1a7147e6a9479c669f082dd74349c6bcb4b (diff) | |
loplugin:simplifybool a little more aggressive
with expressions like !(a && b)
Change-Id: Id2acec2a8d0eaaa8e5e37dbd2cae7281be36572e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100040
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'desktop/source/deployment')
| -rw-r--r-- | desktop/source/deployment/registry/package/dp_package.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/desktop/source/deployment/registry/package/dp_package.cxx b/desktop/source/deployment/registry/package/dp_package.cxx index 890fa3612a42..a6d099db9f5c 100644 --- a/desktop/source/deployment/registry/package/dp_package.cxx +++ b/desktop/source/deployment/registry/package/dp_package.cxx @@ -636,7 +636,7 @@ bool BackendImpl::PackageImpl::checkLicense( OUString sLicense = getTextFromURL(xCmdEnv, sHref); ////determine who has to agree to the license //check correct value for attribute - if ( ! (simplLicAttr->acceptBy == "user" || simplLicAttr->acceptBy == "admin")) + if ( simplLicAttr->acceptBy != "user" && simplLicAttr->acceptBy != "admin") throw css::deployment::DeploymentException( "Could not obtain attribute simple-license@accept-by or it has no valid value", nullptr, Any()); |