[geos-devel] FIXED: problem with GEOS buffer(0) -- JTS works

[Paul Ramsey]

Tag it and bag it...
P

On 22-Aug-05, at 6:44 AM, strk at refractions.net wrote:

> Found and fixed the bug (BufferSubgraphGT).
>
> It was StrictWeakOrdering predicate again, like the one
> in DepthSegmentLT which triggered release 2.1.3.
>
> I think this calls for a 2.1.4
>
> To avoid repeating this any further I've taken a look
> at all sort() invocations, making sure used comparators
> implemented the required semantic. I've changed 3 more
> predicates this way.
>
> Following is a patch for BufferSubgraphGT only
> (which is enough to fix Dave's test case).
>
> For full patch extraction refer to the 'branch-2-1'
> branch of cvs.
>
> --strk;
>
> Index: source/operation/buffer/BufferBuilder.cpp
> ===================================================================
> diff -U2 -r1.27 BufferBuilder.cpp
> --- source/operation/buffer/BufferBuilder.cpp   19 May 2005  
> 10:29:28 -0000
> 1.27
> +++ source/operation/buffer/BufferBuilder.cpp   22 Aug 2005  
> 13:31:02 -0000
> @@ -262,5 +262,5 @@
>
>  bool BufferSubgraphGT(BufferSubgraph *first, BufferSubgraph  
> *second) {
> -       if (first->compareTo(second)>=0)
> +       if (first->compareTo(second)>0)
>                 return true;
>         else
> _______________________________________________
> geos-devel mailing list
> geos-devel at geos.refractions.net
> http://geos.refractions.net/mailman/listinfo/geos-devel
>