diff options
author | Jochen Nitschke <j.nitschke+logerrit@ok.de> | 2018-04-29 23:35:51 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-05-01 09:16:13 +0200 |
commit | af94177d64828d227fd4008ec797378715282192 (patch) | |
tree | 7a4f76843d8ec2e87af89f1d62eb159d1c186027 /forms | |
parent | aea66f8a3d5757e073b9bdcf1ff240371777314e (diff) |
improve AttributeState compare operator
guess this was the original intention, else the nullptr checks
would be pointless
Change-Id: Ifb41b125c6d297c76c1d1b5c06c8f9f127b8e613
Reviewed-on: https://gerrit.libreoffice.org/53644
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'forms')
-rw-r--r-- | forms/source/richtext/rtattributes.hxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/forms/source/richtext/rtattributes.hxx b/forms/source/richtext/rtattributes.hxx index a47c5e22abd3..6f79286f653f 100644 --- a/forms/source/richtext/rtattributes.hxx +++ b/forms/source/richtext/rtattributes.hxx @@ -115,10 +115,10 @@ namespace frm if ( !pItemHandleItem && _rRHS.pItemHandleItem ) return false; - if ( !pItemHandleItem && !_rRHS.pItemHandleItem ) + if ( pItemHandleItem == _rRHS.pItemHandleItem ) return true; - return pItemHandleItem == _rRHS.pItemHandleItem; + return *pItemHandleItem == *_rRHS.pItemHandleItem; } class IMultiAttributeDispatcher |