mirror of https://github.com/CGAL/cgal
Remove some assertions, as they must not pop-up, since they are
used by Cartesian_converter/Homogeneous_converter, where we
absolutely must not throw.
TODO : find a way to keep the assertion while having the
converters still work. (should types come with their
own conversions, like they all have .bbox() ?)
This commit is contained in:
parent
15929ddde9
commit
9424d0e9d8
|
|
@ -52,9 +52,10 @@ public:
|
|||
Iso_cuboidC3(const Point_3 &p, const Point_3 &q, int)
|
||||
: base(p, q)
|
||||
{
|
||||
CGAL_kernel_assertion(p.x()<=q.x());
|
||||
CGAL_kernel_assertion(p.y()<=q.y());
|
||||
CGAL_kernel_assertion(p.z()<=q.z());
|
||||
// I have to remove the assertions, because of Cartesian_converter.
|
||||
// CGAL_kernel_assertion(p.x()<=q.x());
|
||||
// CGAL_kernel_assertion(p.y()<=q.y());
|
||||
// CGAL_kernel_assertion(p.z()<=q.z());
|
||||
}
|
||||
|
||||
Iso_cuboidC3(const Point_3 &p, const Point_3 &q)
|
||||
|
|
|
|||
|
|
@ -52,7 +52,8 @@ public:
|
|||
Iso_rectangleC2(const Point_2 &p, const Point_2 &q, int)
|
||||
: base(p, q)
|
||||
{
|
||||
CGAL_kernel_assertion(p<=q);
|
||||
// I have to remove the assertions, because of Cartesian_converter.
|
||||
// CGAL_kernel_assertion(p<=q);
|
||||
}
|
||||
|
||||
const Point_2 & min() const
|
||||
|
|
|
|||
|
|
@ -1955,8 +1955,9 @@ namespace CartesianKernelFunctors {
|
|||
Iso_rectangle_2
|
||||
operator()(const Point_2& p, const Point_2& q, int) const
|
||||
{
|
||||
CGAL_kernel_assertion(p.x()<=q.x());
|
||||
CGAL_kernel_assertion(p.y()<=q.y());
|
||||
// I have to remove the assertions, because of Cartesian_converter.
|
||||
// CGAL_kernel_assertion(p.x()<=q.x());
|
||||
// CGAL_kernel_assertion(p.y()<=q.y());
|
||||
return Rep(p, q, 0);
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -51,8 +51,9 @@ public:
|
|||
Iso_rectangleH2(const Point_2& p, const Point_2& q, int)
|
||||
: base(p, q)
|
||||
{
|
||||
CGAL_kernel_assertion(p.x()<=q.x());
|
||||
CGAL_kernel_assertion(p.y()<=q.y());
|
||||
// I have to remove the assertions, because of Homogeneous_converter.
|
||||
// CGAL_kernel_assertion(p.x()<=q.x());
|
||||
// CGAL_kernel_assertion(p.y()<=q.y());
|
||||
}
|
||||
|
||||
const Point_2 & min() const;
|
||||
|
|
|
|||
|
|
@ -2523,8 +2523,9 @@ namespace HomogeneousKernelFunctors {
|
|||
Iso_rectangle_2
|
||||
operator()(const Point_2& p, const Point_2& q, int) const
|
||||
{
|
||||
CGAL_kernel_assertion(p.x()<=q.x());
|
||||
CGAL_kernel_assertion(p.y()<=q.y());
|
||||
// I have to remove the assertions, because of Cartesian_converter.
|
||||
// CGAL_kernel_assertion(p.x()<=q.x());
|
||||
// CGAL_kernel_assertion(p.y()<=q.y());
|
||||
return Rep(p, q, 0);
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue