Skip to content

Circular_kernel_2: Cleanup#5735

Merged
sloriot merged 3 commits intoCGAL:masterfrom
afabri:CK_2-cleanup-GF
Aug 12, 2021
Merged

Circular_kernel_2: Cleanup#5735
sloriot merged 3 commits intoCGAL:masterfrom
afabri:CK_2-cleanup-GF

Conversation

@afabri
Copy link
Member

@afabri afabri commented May 28, 2021

Summary of Changes

No longer mention Lazy_circular_kernel_2 in the test. We leave it in benchmark though.

Release Management

Copy link
Member

@mglisse mglisse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that I am not familiar at all with this package, I was probably just looking for the string "Lazy".
Maybe NT could be Exact_rational.

@afabri
Copy link
Member Author

afabri commented May 28, 2021

I just also cleaned up the benchmarks a little bit.

When grepping for "lazy" I discovered the file CGAL/Simple_circular_kernel_2.h which neither in an example nor in a test.

@maxGimeno
Copy link
Contributor

@sloriot sloriot self-assigned this Aug 12, 2021
@sloriot sloriot merged commit 763a1d1 into CGAL:master Aug 12, 2021
@sloriot sloriot deleted the CK_2-cleanup-GF branch August 12, 2021 08:02
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.

Lazy_circular_kernel_2

6 participants