fix ScDetOpList::operator==

should be comparing the values of the elements, not the pointers.

Regression from
    commit 92744735
    Date:   Sat Sep 12 16:49:29 2015 -0600
    replace boost::ptr_container with std::container<std::unique_ptr>

noticed while doing some other plugin work

Change-Id: I96230e52571e9a96e1c96f53f61bbc5afe902b31
Reviewed-on: https://gerrit.libreoffice.org/68738
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

sc/inc/detdata.hxx

@@ -51,6 +51,9 @@ public:
 
     // for UpdateRef:
     void                SetPos(const ScAddress& rNew)   { aPos=rNew; }
+
+    bool operator==     ( const ScDetOpData& r ) const
+                        { return eOperation == r.eOperation && aPos == r.aPos; }
 };
 
 //  list of operators

sc/source/core/tool/detdata.cxx

@@ -79,7 +79,7 @@ bool ScDetOpList::operator==( const ScDetOpList& r ) const
     size_t nCount = Count();
     bool bEqual = ( nCount == r.Count() );
     for (size_t i=0; i<nCount && bEqual; i++)       // order has to be the same
-        if ( aDetOpDataVector[i] != r.aDetOpDataVector[i] )    // entries are different ?
+        if ( !(*aDetOpDataVector[i] == *r.aDetOpDataVector[i]) )    // entries are different ?
             bEqual = false;
 
     return bEqual;