Ensure iterators are value-initialized

...instead of merely default-initialized, to avoid undefined behavior in
enclosing classes' copy ctors/assignment ops.  (Cf. resolution of
<http://www.open-std.org/JTC1/SC22/WG21/docs/lwg-closed.html#408> DR 408 "Is
vector<reverse_iterator<char*> > forbidden?")

In return, remove the enclosing classes' user defined assignment ops that do
explicitly exactly what the default ones would do anyway.

Change-Id: If1bb1aea56dd48da873a92a8fa30dedea844740c

1 files changed, 0 insertions, 16 deletions

diff --git a/sc/source/core/data/mtvelements.cxx b/sc/source/core/data/mtvelements.cxx
index 57573a5a1480..efe07cff5ad7 100644
--- a/sc/source/core/data/mtvelements.cxx
+++ b/sc/source/core/data/mtvelements.cxx
@@ -25,22 +25,6 @@ CellTextAttr::CellTextAttr(sal_uInt16 nTextWidth, sal_uInt8 nScriptType) :
     mnTextWidth(nTextWidth),
     mnScriptType(nScriptType) {}
 
-ColumnBlockPosition& ColumnBlockPosition::operator= (const ColumnBlockPosition& r)
-{
-    miBroadcasterPos = r.miBroadcasterPos;
-    miCellTextAttrPos = r.miCellTextAttrPos;
-    miCellPos = r.miCellPos;
-    return *this;
-}
-
-ColumnBlockConstPosition& ColumnBlockConstPosition::operator= (const ColumnBlockConstPosition& r)
-{
-    miBroadcasterPos = r.miBroadcasterPos;
-    miCellTextAttrPos = r.miCellTextAttrPos;
-    miCellPos = r.miCellPos;
-    return *this;
-}
-
 ColumnBlockPositionSet::ColumnBlockPositionSet(ScDocument& rDoc) : mrDoc(rDoc) {}
 
 ColumnBlockPosition* ColumnBlockPositionSet::getBlockPosition(SCTAB nTab, SCCOL nCol)