diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2013-08-13 13:50:12 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2013-08-13 13:55:04 +0200 |
| commit | a745bfdb7843a216dae51e4cd9484d950bac13e2 (patch) | |
| tree | 490619d6b838b905b0bf69a645cd44c8847e3e5d /sc/inc | |
| parent | 1be8e912ba8a7d1acaf40d5a8597421c104ab39c (diff) | |
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
Diffstat (limited to 'sc/inc')
| -rw-r--r-- | sc/inc/mtvelements.hxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sc/inc/mtvelements.hxx b/sc/inc/mtvelements.hxx index 443eb6af0339..46a5d409e4d9 100644 --- a/sc/inc/mtvelements.hxx +++ b/sc/inc/mtvelements.hxx @@ -107,7 +107,7 @@ struct ColumnBlockPosition CellTextAttrStoreType::iterator miCellTextAttrPos; CellStoreType::iterator miCellPos; - ColumnBlockPosition& operator= (const ColumnBlockPosition& r); + ColumnBlockPosition(): miCellPos() {} }; struct ColumnBlockConstPosition @@ -116,7 +116,7 @@ struct ColumnBlockConstPosition CellTextAttrStoreType::const_iterator miCellTextAttrPos; CellStoreType::const_iterator miCellPos; - ColumnBlockConstPosition& operator= (const ColumnBlockConstPosition& r); + ColumnBlockConstPosition(): miCellPos() {} }; class ColumnBlockPositionSet |