[geos-devel] GEOS and VC++ 2005

[Artem Pavlenko]

Hi strk,

See below,
> anther way to define
> container-related things, which seems to me
> more consistent with C++ practice:
>
> class EdgeIntersectionList {
> public:
> 	typedef set<...> container;
> 	typedef container::iterator iterator;
> 	typedef container::const_iterator const_iterator;
> ...
> }
>   
Why don't you go one step  further and make templated version e.g 
something like that:

template <typename T>
class EdgeIntersectionList
{
         typedef T container;
         ........................
         ........................   
};
typedef EdgeIntersectionList< std::set <...> > MyEdgeIntersectionList
I think this would be more flexible.

> Do you think we should use this format for all classes ?
>   
Yes.
> I'd say we do, unless there are known platform on which
> this is unsupported.
>   
Do mean compiler support?  I can not see any problems as this is a 
standard c++.
Cheers
Artem