diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-08-06 13:32:43 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-08-14 18:03:06 +0200 |
| commit | e67657d5211f6e95ddf8bd621108608573b00d5d (patch) | |
| tree | 66724101dbd95721714bd40fcb4861663432774c /canvas | |
| parent | tdf#135550 FmXListBoxCell does need to implement XListBox after all (diff) | |
| download | core-e67657d5211f6e95ddf8bd621108608573b00d5d.tar.gz core-e67657d5211f6e95ddf8bd621108608573b00d5d.zip | |
loplugin:simplifybool more
look for expressions like
!(a && !b)
which can be expanded out
Change-Id: I72515a9638762b050f9a258c08da39ebfa2ef8e7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100579
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'canvas')
| -rw-r--r-- | canvas/source/cairo/cairo_spritehelper.cxx | 2 | ||||
| -rw-r--r-- | canvas/source/vcl/spritehelper.cxx | 3 |
2 files changed, 2 insertions, 3 deletions
diff --git a/canvas/source/cairo/cairo_spritehelper.cxx b/canvas/source/cairo/cairo_spritehelper.cxx index 667f1ba249ba..1fe9eb9152e0 100644 --- a/canvas/source/cairo/cairo_spritehelper.cxx +++ b/canvas/source/cairo/cairo_spritehelper.cxx @@ -83,7 +83,7 @@ namespace cairocanvas const double fAlpha( getAlpha() ); const ::basegfx::B2DHomMatrix aTransform( getTransformation() ); - if( !(isActive() && !::basegfx::fTools::equalZero( fAlpha )) ) + if( !isActive() || ::basegfx::fTools::equalZero( fAlpha ) ) return; SAL_INFO( "canvas.cairo", "CanvasCustomSprite::redraw called"); diff --git a/canvas/source/vcl/spritehelper.cxx b/canvas/source/vcl/spritehelper.cxx index df8550c6c9f2..e52bb211020c 100644 --- a/canvas/source/vcl/spritehelper.cxx +++ b/canvas/source/vcl/spritehelper.cxx @@ -98,8 +98,7 @@ namespace vclcanvas const double fAlpha( getAlpha() ); - if( !(isActive() && - !::basegfx::fTools::equalZero( fAlpha )) ) + if( !isActive() || ::basegfx::fTools::equalZero( fAlpha ) ) return; const Point aEmptyPoint; |