Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary of Changes
This addresses issue #9084. In particular, I replaced
CartesianwithSimple_cartesianin all traits.I also defined Multiplicity to be
std::size_tin all traits. In addition, I replacedtypedefwithusingand properly indented and cleaned up the code in the source files I touched. I simplified the code that approximates a point of the traits classesArr_linear_traits_2,Arr_segment_traits_2, andArr_non_caching_segment_basic_traits_2.h. I did not use the GAL::Cartesian_coverter (mentioned in issue #9084) because the code is now simple as it is(just one line in each traits class).I also added a missing
inlinein draw_arrangement_2. This is unrelated to the above.Release Management