Yes, It looks like that you did not change the EnableCheckFactory, while 
you did this for the speed improvements. The necessary changes for the 
speed improvements are also not in SelectionManager, i.e. the XXXCount 
methods.

I think we leave it as it is right now (due to some additional changes 
of EnableCheck).

stefan

Larry Becker wrote:
> Hi Stefan,
> 
>   So did I apply the same fix to SkyJUMP as I did to OpenJUMP?  I'm 
> guessing that I didn't.  Do we need to back out that change and live 
> with the performance problems (for million point tasks)?
> 
> Larry
> 
> On Wed, Mar 19, 2008 at 3:21 PM, Stefan Steiniger <[EMAIL PROTECTED] 
> <mailto:[EMAIL PROTECTED]>> wrote:
> 
>     Hei Larry,
> 
>     the error does not appear in SkyJUMP :)
> 
>     stefan
> 
>     Larry Becker wrote:
>      > Stefan,
>      >
>      >   The sourceforge sources are up to date for SkyJUMP.
>      >
>      > Larry
>      >
>      > On Fri, Mar 14, 2008 at 12:25 PM, Stefan Steiniger
>     <[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>
>      > <mailto:[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>>> wrote:
>      >
>      >     Hei Larry,
>      >
>      >
>      >      > I guess you are asking me because I'm the culprit who
>      >      > did the tuning on selection size?  :-)  I'll take a look
>     at the code
>      >      > when I get a chance.  Haven't been doing much JUMP programming
>      >     lately.
>      >
>      >     ok ;)
>      >     But one question.
>      >
>      >     Are the SkyJUMP sources on sourceforge the actual ones? If
>     yes I can
>      >     check on my own and tell you.
>      >
>      >     stefan
>      >

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to