mirror of https://github.com/CGAL/cgal
updated
This commit is contained in:
parent
30f9bd1157
commit
2dc85e03f4
|
|
@ -19,49 +19,59 @@
|
||||||
- Test all landmark sub-strategies
|
- Test all landmark sub-strategies
|
||||||
|
|
||||||
- Get rid of the prefix 'get_' from all functions and methods:
|
- Get rid of the prefix 'get_' from all functions and methods:
|
||||||
get_bbox
|
get_all_inner_nodes()
|
||||||
|
get_bbox()
|
||||||
Bezier_point_2.h,
|
Bezier_point_2.h,
|
||||||
Bezier_x_monotone_2.h)
|
Bezier_x_monotone_2.h)
|
||||||
get_coefficient
|
get_boundary_in_x()
|
||||||
|
get_boundary_in_y()
|
||||||
|
get_coefficient()
|
||||||
Bezier_cache.h
|
Bezier_cache.h
|
||||||
Bezier_curve_2.h
|
Bezier_curve_2.h
|
||||||
Rational_arc_2.h
|
Rational_arc_2.h
|
||||||
CORE_algebraic_number_traits.h
|
CORE_algebraic_number_traits.h
|
||||||
get_index
|
get_data()
|
||||||
|
Td_dag.h
|
||||||
|
get_depth_threshold()
|
||||||
|
Trapezoidal_decomposition_2.h
|
||||||
|
get_double()
|
||||||
|
Arr_lm_random_generator.h
|
||||||
|
Arr_point_location/Arr_lm_random_generator.h
|
||||||
|
get_index()
|
||||||
Arr_circle_segment_traits_2.h
|
Arr_circle_segment_traits_2.h
|
||||||
Arr_conic_traits_2.h
|
Arr_conic_traits_2.h
|
||||||
get_data
|
get_intersections()
|
||||||
Td_dag.h
|
get_lb()
|
||||||
get_level
|
Td_X_trapezoid.h
|
||||||
|
Trapezoidal_decomposition_2.h
|
||||||
|
get_level()
|
||||||
Arr_qdx_topology_traits_2_impl.h
|
Arr_qdx_topology_traits_2_impl.h
|
||||||
Arr_torus_topology_traits_2_impl.h
|
Arr_torus_topology_traits_2_impl.h
|
||||||
get_lb
|
get_lt()
|
||||||
Td_X_trapezoid.h
|
Td_X_trapezoid.h
|
||||||
Trapezoidal_decomposition_2.h
|
Trapezoidal_decomposition_2.h
|
||||||
get_lt
|
get_mode()
|
||||||
Td_X_trapezoid.h
|
get_node()
|
||||||
Trapezoidal_decomposition_2.h
|
|
||||||
get_node
|
|
||||||
Td_X_trapezoid.h
|
Td_X_trapezoid.h
|
||||||
Trapezoidal_decomposition_2.h
|
Trapezoidal_decomposition_2.h
|
||||||
get_parent
|
get_originator()
|
||||||
|
get_painter()
|
||||||
|
get_parent()
|
||||||
Arr_trapezoid_ric_pl_impl.h
|
Arr_trapezoid_ric_pl_impl.h
|
||||||
Trapezoidal_decomposition_2.h
|
Trapezoidal_decomposition_2.h
|
||||||
Arr_trapezoid_ric_point_location.h
|
Arr_trapezoid_ric_point_location.h
|
||||||
get_rb
|
get_rb()
|
||||||
Td_X_trapezoid.h
|
Td_X_trapezoid.h
|
||||||
Trapezoidal_decomposition_2.h
|
Trapezoidal_decomposition_2.h
|
||||||
get_rt
|
get_rt()
|
||||||
Td_X_trapezoid.h
|
Td_X_trapezoid.h
|
||||||
Trapezoidal_decomposition_2.h
|
Trapezoidal_decomposition_2.h
|
||||||
get_depth_threshold
|
get_size_threshold()
|
||||||
Trapezoidal_decomposition_2.h
|
Trapezoidal_decomposition_2.h
|
||||||
get_size_threshold
|
get_t_at_x()
|
||||||
Trapezoidal_decomposition_2.h
|
get_t_at_y()
|
||||||
get_y
|
get_traits()
|
||||||
get_double
|
get_vertical_tangencies()
|
||||||
Arr_lm_random_generator.h
|
|
||||||
Arr_point_location/Arr_lm_random_generator.h
|
|
||||||
|
|
||||||
- Add insert_* functions back for backward compatibility.
|
- Add insert_* functions back for backward compatibility.
|
||||||
Keep the old interface for backward complatibility (at least for CGAL 3.4,
|
Keep the old interface for backward complatibility (at least for CGAL 3.4,
|
||||||
|
|
@ -77,16 +87,6 @@
|
||||||
operation to perform (just like we do now in insert() ).
|
operation to perform (just like we do now in insert() ).
|
||||||
|
|
||||||
- Spherical arcs
|
- Spherical arcs
|
||||||
- Rewrite the spherical helper class for the vertical-decomposition sweep-
|
|
||||||
line visitor (current one is not good enough).
|
|
||||||
Regarding the helper class for the spherical case, we currently return the
|
|
||||||
"north face" in case we "see" nothing above - which is OK - but we also
|
|
||||||
return the "north face" in case we see nothing from *below*. This is wrong:
|
|
||||||
we should really keep track of the "south face" and return it as the
|
|
||||||
"bottom" object. To conclude, this helper needs more work. It is not top
|
|
||||||
priority, but the quick-and-dirty helper that I wrote needs to become a bit
|
|
||||||
more complicated ...
|
|
||||||
|
|
||||||
- The insertion helper and the batched point-location helper seem to have
|
- The insertion helper and the batched point-location helper seem to have
|
||||||
been copied from the planar helpers with minor changes. That is, I
|
been copied from the planar helpers with minor changes. That is, I
|
||||||
expect to face problem while aggregately inserting new curves into a
|
expect to face problem while aggregately inserting new curves into a
|
||||||
|
|
@ -150,3 +150,24 @@
|
||||||
Now zoom in, and oops, you see the two input objects again.
|
Now zoom in, and oops, you see the two input objects again.
|
||||||
I would expect to zoom into the intersection.
|
I would expect to zoom into the intersection.
|
||||||
|
|
||||||
|
- The arrangement demo fails with conics.
|
||||||
|
|
||||||
|
- There is a warning coming from Cartesian/function_objects.h line 3113
|
||||||
|
about returning a reference to a temporary variable (which seems to be
|
||||||
|
a good warning; perhaps it is the source of some segmentation faults we
|
||||||
|
encounter). A "kernel expert" should look into this.
|
||||||
|
- In the example dual_lines.cpp we use the function rand().
|
||||||
|
We should include <stdlib.h> to avoid a compilation error.
|
||||||
|
- The example circular_line_arcs.cpp does not compile under many platforms,
|
||||||
|
or at least issues warnings about returning a reference to temporary
|
||||||
|
variable. Can you prompt the INRIA people to fix this?
|
||||||
|
(I personally think that we should remove this example from our test-suite,
|
||||||
|
but if the INRIA people believe that it's place is there, they should at
|
||||||
|
least properly maintain it ...)
|
||||||
|
- There is a problem with the examples that use CORE on Darwin (platform #10).
|
||||||
|
Other Darwin platforms seem fine. Do we want to investigate? (perhaps it's
|
||||||
|
just something wrong with the installation ...)
|
||||||
|
- There is a segmentation fault in the examples that use overlay on platforms
|
||||||
|
#10 and #12. Here I don't have a clue ...
|
||||||
|
- There are still some mysterious crashes of the test-suite on some platforms.
|
||||||
|
Again, I don't have a clue ...
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue