[Libreoffice-commits] core.git: sw/source

[Mike Kaganski]

 sw/source/core/crsr/pam.cxx |    4 ----
 1 file changed, 4 deletions(-)

New commits:
commit 202943e49a680b1032bbf45aaa4b0a73dc548867
Author: Mike Kaganski <mike.kaganski at collabora.com>
Date:   Fri Nov 18 10:32:03 2016 +0300

    Remove redundant comparisons
    
    Comparing IndexRegs is included into SwIndex comparison operators
    (see sw/inc/index.hxx), so no need to do it also explicitly.
    
    Change-Id: I7e3ea8ab9b8f852cd7e6f0d90afa26e75c734652
    Reviewed-on: https://gerrit.libreoffice.org/30951
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/sw/source/core/crsr/pam.cxx b/sw/source/core/crsr/pam.cxx
index a71e910..d5cf3da 100644
--- a/sw/source/core/crsr/pam.cxx
+++ b/sw/source/core/crsr/pam.cxx
@@ -160,16 +160,12 @@ bool SwPosition::operator>=(const SwPosition &rPos) const
 bool SwPosition::operator==(const SwPosition &rPos) const
 {
     return (nNode == rPos.nNode)
-        // GetIndexReg may be null for FLY_AT_PARA frame anchor position
-        && (nContent.GetIdxReg() == rPos.nContent.GetIdxReg())
         && (nContent == rPos.nContent);
 }
 
 bool SwPosition::operator!=(const SwPosition &rPos) const
 {
     return (nNode != rPos.nNode)
-        // GetIndexReg may be null for FLY_AT_PARA frame anchor position
-        || (nContent.GetIdxReg() != rPos.nContent.GetIdxReg())
         || (nContent != rPos.nContent);
 }