diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2024-11-07 00:29:51 +0500 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2024-11-06 23:33:53 +0100 |
| commit | 2ee7745225fb2c5340dc8eddd803a8de7eeb6865 (patch) | |
| tree | 6a1691d3f49ded36bf2ae611798d6875b778ae1d /editeng/source | |
| parent | dcf3a85e6e5d4c7761d73a9f018c0c09862351ce (diff) | |
aGraphic.IsNone was checked above
It seems that the else branch position wasn't adjusted in commit
7a8ed362eb163ac15a000ba1cfc74b58315800a1 ([API CHANGE] revert and
deprecate *BackGraphicURL add *BackGraphic, 2018-05-07).
Change-Id: Ib4b68ee813539b4d62ffd8664d496467d7789547
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/176158
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Jenkins
Diffstat (limited to 'editeng/source')
| -rw-r--r-- | editeng/source/items/frmitems.cxx | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/editeng/source/items/frmitems.cxx b/editeng/source/items/frmitems.cxx index ef60738977fe..96db6916d454 100644 --- a/editeng/source/items/frmitems.cxx +++ b/editeng/source/items/frmitems.cxx @@ -4396,15 +4396,15 @@ bool SvxBrushItem::PutValue( const uno::Any& rVal, sal_uInt8 nMemberId ) ApplyGraphicTransparency_Impl(); xOldGrfObj.reset(); - if (aGraphic.IsNone()) - { - eGraphicPos = GPOS_NONE; - } - else if (eGraphicPos == GPOS_NONE) + if (eGraphicPos == GPOS_NONE) { eGraphicPos = GPOS_MM; } } + else + { + eGraphicPos = GPOS_NONE; + } } break; |