luke/klayout
1
0
Fork 0
mirror of https://github.com/KLayout/klayout.git synced 2026-09-04 08:43:42 +02:00
Code Issues Packages Projects Releases Wiki Activity

Fixed #277 (min_coherence not recognized by region size)

Browse Source
This commit is contained in:
Matthias Koefferlein
2019-06-17 21:40:37 +02:00
parent f59e49d678
commit 303cda6981
11 changed files with 75 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/db/unit_tests/dbDeepRegionTests.cc
+27
View File
@@ -1406,3 +1406,30 @@ TEST(101_DeepFlatCollaboration)
db::compare_layouts (_this, target, tl::testsrc () + "/testdata/algo/deep_region_au101.gds");
}
TEST(issue_277)
{
db::Layout ly;
db::cell_index_type top_cell_index = ly.add_cell ("TOP");
db::Cell &top_cell = ly.cell (top_cell_index);
unsigned int l1 = ly.insert_layer ();
db::Shapes &s = top_cell.shapes (l1);
s.insert (db::Box (0, 0, 400, 400));
s.insert (db::Box (400, 400, 800, 800));
db::DeepShapeStore dss;
db::Region r (db::RecursiveShapeIterator (ly, top_cell, l1), dss);
EXPECT_EQ (r.sized (1).merged (false, 1).to_string (), "");
r.set_min_coherence (true);
EXPECT_EQ (r.sized (1).merged (false, 1).to_string (), "(399,399;399,401;401,401;401,399)");
r.merge ();
EXPECT_EQ (r.sized (1).merged (false, 1).to_string (), "(399,399;399,401;401,401;401,399)");
r.set_min_coherence (false); // needs to merge again
EXPECT_EQ (r.sized (1).merged (false, 1).to_string (), "");
}
src/db/unit_tests/dbRegion.cc
+18
View File
@@ -1397,3 +1397,21 @@ TEST(issue_228)
EXPECT_EQ (r.selected_interacting (rr).to_string (), r.to_string ());
EXPECT_EQ (rr.selected_interacting (r).to_string (), rr.to_string ());
}
TEST(issue_277)
{
db::Region r;
r.insert (db::Box (0, 0, 400, 400));
r.insert (db::Box (400, 400, 800, 800));
EXPECT_EQ (r.sized (1).merged (false, 1).to_string (), "");
r.set_min_coherence (true);
EXPECT_EQ (r.sized (1).merged (false, 1).to_string (), "(399,399;399,401;401,401;401,399)");
r.merge ();
EXPECT_EQ (r.sized (1).merged (false, 1).to_string (), "(399,399;399,401;401,401;401,399)");
r.set_min_coherence (false); // needs to merge again
EXPECT_EQ (r.sized (1).merged (false, 1).to_string (), "");
}