diff options
| author | Bogdan Buzea <buzea.bogdan@libreoffice.org> | 2024-10-20 08:09:04 +0200 |
|---|---|---|
| committer | Xisco Fauli <xiscofauli@libreoffice.org> | 2024-10-20 10:30:22 +0200 |
| commit | ca48b37054d68429e323235ff8dda86e85d02aa5 (patch) | |
| tree | a2c0732977411b21c22837ce6b000d6816b251fe | |
| parent | faafdec09ae2b17aa62d091f76e0cca1b0867d17 (diff) | |
tdf#163486: PVS: Identical branches
V1037 Two or more case-branches perform the same actions. Check lines: 856, 860
V1037 Two or more case-branches perform the same actions. Check lines: 918, 923, 958
Change-Id: I2ef4a1cbcbbc54496b896153f08ebd0049c4dfc8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175232
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
| -rw-r--r-- | vcl/source/outdev/transparent.cxx | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/vcl/source/outdev/transparent.cxx b/vcl/source/outdev/transparent.cxx index 1c8b116408b5..c98f84823c67 100644 --- a/vcl/source/outdev/transparent.cxx +++ b/vcl/source/outdev/transparent.cxx @@ -853,9 +853,6 @@ void ImplConvertTransparentAction( GDIMetaFile& o_rMtf, break; case MetaActionType::BMPEXSCALE: - aBmpEx = static_cast<const MetaBmpExScaleAction&>(rAct).GetBitmapEx(); - break; - case MetaActionType::BMPEXSCALEPART: aBmpEx = static_cast<const MetaBmpExScaleAction&>(rAct).GetBitmapEx(); break; @@ -915,11 +912,8 @@ bool ImplIsNotTransparent( const MetaAction& rAct, const OutputDevice& rOut ) switch( rAct.GetType() ) { case MetaActionType::POINT: - if( !bLineTransparency ) - bRet = true; - break; - case MetaActionType::LINE: + case MetaActionType::POLYLINE: if( !bLineTransparency ) bRet = true; break; @@ -954,11 +948,6 @@ bool ImplIsNotTransparent( const MetaAction& rAct, const OutputDevice& rOut ) bRet = true; break; - case MetaActionType::POLYLINE: - if( !bLineTransparency ) - bRet = true; - break; - case MetaActionType::POLYGON: if( !bLineTransparency || !bFillTransparency ) bRet = true; |