Skip to Content.
Sympa Menu

cgal-discuss - Re: [cgal-discuss] Some CGAL 4.4 Bug Fixes

Subject: CGAL users discussion list

List archive

Re: [cgal-discuss] Some CGAL 4.4 Bug Fixes


Chronological Thread 
  • From: Andreas Fabri <>
  • To:
  • Subject: Re: [cgal-discuss] Some CGAL 4.4 Bug Fixes
  • Date: Thu, 24 Apr 2014 17:32:22 +0200
  • Organization: GeometryFactory


Hi Jeff,

thank you for your bug fixes. Especially the kinetic data structure
bug bothered us for quite a while. My more compact solution is to copy the

typedef ... Root_stack;

into the nested class.

It will make it into the bug fix release.

Just out of curiosity: What do you with the kinetic data structures?


best,

andreas

On 19/04/2014 23:37, Jeffrey Bush wrote:
include\CGAL\Kinetic\Delaunay_triangulation_3.h@90-91:
- typedef internal::Delaunay_3_edge_flip_event<This_DT3, Root_stack>
Edge_flip;
- typedef typename internal::Delaunay_3_facet_flip_event<This_DT3,
Root_stack> Facet_flip;
+ typedef internal::Delaunay_3_edge_flip_event<This_DT3, typename
TraitsT::Kinetic_kernel::Side_of_oriented_sphere_3::result_type> Edge_flip;
+ typedef typename internal::Delaunay_3_facet_flip_event<This_DT3,
typename TraitsT::Kinetic_kernel::Side_of_oriented_sphere_3::result_type>
Facet_flip;

This caused the error in Delaunay_triangulation_base_3.h that many people
have complained about. Apparently in the definition here, the Root_stack was
picking up some other random Root_stack and not the one defined shortly
above it. Fix was to use the explicit type instead of Root_stack. There are
probably more condensed versions of it that work, but this one was the first
I tried and it worked so I stuck with it. It*should* work fine in other
compilers too.

Thanks!
Jeff

--
Andreas Fabri, PhD
Chief Officer, GeometryFactory
Editor, The CGAL Project

phone: +33.492.954.912 skype: andreas.fabri



Archive powered by MHonArc 2.6.18.

Top of Page