[geos-devel] Re: gettimeofday for win32
strk at refractions.net
strk at refractions.net
Fri Dec 3 05:28:41 EST 2004
On Thu, Dec 02, 2004 at 09:22:27AM -0500, Frank Warmerdam wrote:
(...)
> I don't see any VisualStudio directory in the 2.1.0 distribution. Is
> it only available in CVS? I don't really want a project/workspace
> file anyways, though I suppose if it worked smoothly enough it
> would do.
I confirm it's not included in the distribution.
Paul, should it ?
(...)
> o The definition of int64 in platform.h lacks a win32 case.
> o The use of the long long static uid in various items in geom.h caused
> a problem. I just hacked out the UIDs as they don't seem to have any
> purpose.
That is mainly Martin's way of versioning classes.
Anyway a 64-bit int type is needed for the DoubleBits class, better
to handle win32 case in platform.h.
> o The base class toVector() returns a const result while the overrides return
> non-const. I had to change the base to non-const as well, though I am
> not clear what the original intent was.
We should probably have two methods:
const vector<Coordinate>* toVector() const;
vector<Coordinate>* toVector();
--strk;
> o Some problems with time stuff for the profiler. It was your mention of
> this that made me think you were building on win32 as well.
>
> I haven't gone back to try things on VC6 but if I do, i will
> keep the stlport thing in mind. I do my FWTools builds with
> VC6, so if I am to GEOS enable that distribution I will need to
> solve VC6 builds.
>
> Best regards,
> --
> ---------------------------------------+--------------------------------------
> I set the clouds in motion - turn up | Frank Warmerdam, warmerdam at pobox.com
> light and sound - activate the windows | http://pobox.com/~warmerdam
> and watch the world go round - Rush | Geospatial Programmer for Rent
> _______________________________________________
> geos-devel mailing list
> geos-devel at geos.refractions.net
> http://geos.refractions.net/mailman/listinfo/geos-devel
More information about the geos-devel
mailing list