diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-04-19 09:07:27 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-05-30 10:49:51 +0200 |
| commit | d6c32cffb5cc81989b4bb4a221a152bbe607bd98 (patch) | |
| tree | 2c4ef9ec2a201ffcfea145ed5a0b901f2b6853ea /svgio/source/svgreader/svgpatternnode.cxx | |
| parent | 9ab64dc48a6a61edce6ff3724093162ca1cf8331 (diff) | |
loplugin:simplifybool extend to expression like !(a < b || c > d)
mostly to catch stuff from the flatten work, but I think this looks good
in general
Change-Id: I7be5b7bcf1f3d9f980c748ba20793965cef957e7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92493
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'svgio/source/svgreader/svgpatternnode.cxx')
| -rw-r--r-- | svgio/source/svgreader/svgpatternnode.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/svgio/source/svgreader/svgpatternnode.cxx b/svgio/source/svgreader/svgpatternnode.cxx index a72846aacd07..3e7402cb07a3 100644 --- a/svgio/source/svgreader/svgpatternnode.cxx +++ b/svgio/source/svgreader/svgpatternnode.cxx @@ -196,7 +196,7 @@ namespace svgio::svgreader double fTargetWidth(rGeoRange.getWidth()); double fTargetHeight(rGeoRange.getHeight()); - if(!(fTargetWidth > 0.0 && fTargetHeight > 0.0)) + if(fTargetWidth <= 0.0 || fTargetHeight <= 0.0) return; const SvgUnits aPatternUnits(getPatternUnits() ? *getPatternUnits() : objectBoundingBox); @@ -226,7 +226,7 @@ namespace svgio::svgreader rfH /= fTargetHeight; } - if(!(rfW > 0.0 && rfH > 0.0)) + if(rfW <= 0.0 || rfH <= 0.0) return; if(objectBoundingBox == aPatternUnits) |