Boost logo

Geometry :

Subject: [ggl] spatial_index to index
From: Barend Gehrels (Barend.Gehrels)
Date: 2009-06-17 10:03:15


Hi Mateusz,

> I changed the namespace name too (r828), but I'm not sure about it,
> so please consider it as temporary change to be reviewed.
>
> So, now it's ggl::index::rtree, but I'm completely unsure.
> Probably it should be ggl:: so rtree class sits in ggl::rtree?
>

Nice. The namespace index is OK to me. Folder structure is OK to me. The
namespace rtree sounds a littlebit redundant to me.

    typedef typename ggl::index::rtree::rtree<Box, P, int> type;

I would expect an ggl::index::rtree here, . For the implementation, it
might go into "detail::rtree". It is then similar to the algorithms (be
it that they are in ggl::distance, but we don't want to have ::algorithm
there)

Regards, Barend


Geometry list run by mateusz at loskot.net