luke
/
klayout
mirror of https://github.com/KLayout/klayout.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Removing a TODO - that suggestion isn't good as of now

This commit is contained in:
Matthias Koefferlein 2024-05-31 15:59:32 +02:00
parent 2e7e299af9
commit 3902ad6576
2 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/db/db/dbInstances.cc
View File

@ -539,11 +539,6 @@ template class instance_iterator<OverlappingInstanceIteratorTraits>;
// ------------------------------------------------------------------------------------- // -------------------------------------------------------------------------------------
// NormalInstanceIteratorTraits implementation // NormalInstanceIteratorTraits implementation
// TODO: this class could use standard iterators instead of flat
// box tree ones. This potentially saves a sorting step when
// no box trees are needed and order will remain more stable in
// that case.
NormalInstanceIteratorTraits::NormalInstanceIteratorTraits () NormalInstanceIteratorTraits::NormalInstanceIteratorTraits ()
: mp_insts (0) : mp_insts (0)
{ } { }

4
src/db/unit_tests/dbCellTests.cc
View File

@ -998,6 +998,10 @@ TEST(6)
// Note: iterating and replace does not work in non-editable mode // Note: iterating and replace does not work in non-editable mode
if (db::default_editable_mode ()) { if (db::default_editable_mode ()) {
std::vector<db::Instance> insts;
for (db::Cell::const_iterator i = cc.begin (); ! i.at_end (); ++i) {
insts.push_back (*i);
}
for (db::Cell::const_iterator i = cc.begin (); ! i.at_end (); ++i) { for (db::Cell::const_iterator i = cc.begin (); ! i.at_end (); ++i) {
cc.replace_prop_id (*i, i->prop_id () + 1); cc.replace_prop_id (*i, i->prop_id () + 1);
} }