Re: [Lib2geom-devel] Mandatory path stitching

[Krzysztof K. <twe...@gm...>]

A non-contiguous path is practically always an error, so it makes
sense to either prevent it or report it as early as possible. Now the
default behavior is to add those extra segments, which allows sloppily
written code to function as expected, but you can call
setStitchExceptions() to enable the throwing of exceptions whenever a
stitch segment would be appended.

Regards, Krzysztof

2015-03-28 13:09 GMT+01:00 Johan Engelen <jbc...@sw...>:
> On 27-3-2015 20:48, Krzysztof Kosiński wrote:
>> Every method that modifies a Path has an optional argument called
>> "stitching" that determines whether additional segments are added to
>> make the path continuous. By default, stitching is not used. If there
>> are any discontinuities, calling some of Path will cause an exception
>> to be thrown, which in most cases means a crash.
>>
>> I want to make stitching mandatory. Since in some cases you want to be
>> sure that no stitching happens in your code, there will be an extra
>> boolean member variable. When set to true, it will throw an exception
>> whenever you try to insert a discontinuity into the path.
>>
>> Is everyone OK with this idea?
>
> I am not so sure about this.
> It may not be very helpful if 2geom automatically fixes "user errors",
> leading to unexpected results. If you change the default behavior (from
> not stitching to stitching), user code will suddenly behave different
> perhaps without the user knowing (e.g. no compiler error).
> How about making the _choice_ for stitching behavior mandatory?
>
> -Johan
>
>
> ------------------------------------------------------------------------------
> Dive into the World of Parallel Programming The Go Parallel Website, sponsored
> by Intel and developed in partnership with Slashdot Media, is your hub for all
> things parallel software development, from weekly thought leadership blogs to
> news, videos, case studies, tutorials and more. Take a look and join the
> conversation now. http://goparallel.sourceforge.net/
> _______________________________________________
> Lib2geom-devel mailing list
> Lib...@li...
> https://lists.sourceforge.net/lists/listinfo/lib2geom-devel