Skip to content

Conversation

@gdamiand
Copy link
Member

@gdamiand gdamiand commented Jan 8, 2026

Summary of Changes

Correction of two bugs in combinatorial map package:

  • const correctness of Element_topo
  • missing operator[] for dart storage with index

Release Management

  • Affected package(s): Combinatorial maps

@afabri
Copy link
Member

afabri commented Jan 12, 2026

@gdamiand it would make sense to add a test case, because you write it fixes a bug, but the testsuite is green.

@afabri afabri assigned afabri and unassigned afabri Jan 12, 2026
@afabri
Copy link
Member

afabri commented Jan 12, 2026

Also maybe backport to 6.1 ?

@gdamiand
Copy link
Member Author

@gdamiand it would make sense to add a test case, because you write it fixes a bug, but the testsuite is green.

Done.

@sloriot
Copy link
Member

sloriot commented Jan 14, 2026

Successfully tested in CGAL-6.2-Ic-82

@sloriot sloriot merged commit aecdd5a into CGAL:main Jan 14, 2026
9 checks passed
@sloriot sloriot deleted the CMap_bug-gdamiand branch January 14, 2026 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants