Hi all, I just want to remember that I recently pointed out that "Draw polygon" and "draw line" tools have the same functions of "Draw contrained polygon" and "Draw constrained line", also with the usage of the shortcuts. My proposal was to elimitate the original Draw features from the toolbar and save only the constrained ones If you agree to this proposal, it is probabily inot important to solve this java error
regards Peppe --- Gio 18/12/08, Larry Becker <becker.la...@gmail.com> ha scritto: > Da: Larry Becker <becker.la...@gmail.com> > Oggetto: Re: [JPP-Devel] R: R: IndexOutOfBoundsExceptions with Java 1.6.0_11 > A: "OpenJump develop and use" <jump-pilot-devel@lists.sourceforge.net> > Data: Giovedì 18 dicembre 2008, 17:05 > So your solution is blame the user for a Java bug? > > On Thu, Dec 18, 2008 at 9:59 AM, Sunburned Surveyor < > sunburned.surve...@gmail.com> wrote: > > > Larry wrote: "I don't think we need a > workaround, do we?" > > > > I think this bug pointed out a lack of robustness (or > fragility) in > > the MultiClickTool class. I looked at the tool this > morning, and I > > think I have an easy fix that will handle this error > condition > > gracefully. It will also handle other similar error > conditions more > > gracefully. > > > > Here is the fix: > > > > I modified the getShape method in the MultiClickTool > class to throw an > > IllegalStateException with a unique message if the > getShape method is > > called and the user has not made any clicks (the > coordinates ArrayList > > is empty). I then throw this exception from the > method. > > > > The getShape method is called by the drawShapeXOR > method in the > > AbstractCursorTool class. I wrapped the implementation > of this methon > > in a try/catch block. In the catch part of the block I > snag any > > IllegalStateExceptions that are thrown. If the message > of the > > Exception matches the one thrown from the getShape > method of the > > MultiClickTool I get the WorkbenchFrame and warn the > user that they > > need to click on the layer view more than twice before > attempting to > > draw the shape. > > > > This may not be a perfect fit, but it is certainly > better than getting > > a stack trace with an exception from an ArrayList > burried deep in the > > code. If the fix is implemented we will, at a minimum: > > > > - Know that the problem really has to do with an > illegal state in the > > cursor tool (no clicks in the coordinate list). > > - Be able to give the user some reasonable feedback. > > - Will be able to easily catch other > IllegalStateExceptions caused by > > AbstractCursorTool extensions in the future and > respond to them if > > appropriate. > > > > I tried to move the implementation of the drawShapeXOR > method with my > > exception handling directly to the MultiClickTool > class, but that > > causes other problems, since the majority of the > classes "meat" is in > > the AbstractCursorTool class. If there are other ways > to improve my > > fix, I'd like to hear about them. > > > > Any comments? Any objections to my commiting the fix > if there is no > > concerns? > > > > SS > > > > On Thu, Dec 18, 2008 at 6:52 AM, Larry Becker > <becker.la...@gmail.com> > > wrote: > > > Hi Peppe, > > > > > > Your tests seem pretty definitive. We should > probably discourage the use > > of > > > Java 1.6 10 and 11, although they seem to do no > real harm. > > > > > > Looks like there were a ton of changes in _10: > > > http://java.sun.com/javase/6/webnotes/6u10.html > > > > > > The only remaining issue I see is determining if > this (Windows?) Java bug > > > has been reported. I don't think we need a > workaround, do we? > > > > > > Larry > > > > > > On Thu, Dec 18, 2008 at 6:44 AM, Giuseppe Aruta > <giuseppe_ar...@yahoo.it > > > > > > wrote: > > >> > > >> Hi all, > > >> > > >> I did some tests, following the information > of Jukka > > >> > > >> - select either draw polygon or draw > linestring tool (basic, not the > > >> constrained ones) > > >> - RIGHT click on map and get the menu > > >> - LEFT click on map, and there it is (the > error)! > > >> > > >> I tried using > > >> -JRE 1.5.0_12 > > >> -JRE 1.6.0_07 > > >> -JRE 1.6.o_10 > > >> -JRE 1.6.0_11 > > >> > > >> And two version of the softwrare > > >> -OpenJUMP 1.2F (stable realize - zip version) > > >> -openjump-core-1.2-20081218-bin (nigtly > build) > > >> > > >> Windows XP SP3 > > >> ADM Athlon(TM) XP 1800+ > > >> 1.53 Mhz 469 Mb > > >> > > >> -------------------------------------------- > > >> The Test > > >> > > >> OpenJUMP 1.2F (stable realize) > > >> > > >> -JRE 1.5.0_12 OK > > >> -JRE 1.6.0_07 OK > > >> -JRE 1.6.o_10 ERROR > > >> -JRE 1.6.0_11 ERROR > > >> > > >> Openjump-1.2-20081218 (nigtly build) > > >> > > >> -JRE 1.5.0_12 OK > > >> -JRE 1.6.0_07 OK > > >> -JRE 1.6.o_10 ERROR > > >> -JRE 1.6.0_11 ERROR > > >> _____________________________________ > > >> > > >> With this test I can exclude that the problem > can be connected to > > >> something changed in OpenJUMP from ver F to > actual one. > > >> And the problem got birth from jre1.6.0_07 to > jre1.6.0_10 version, which > > >> probabily you know, but now empirically > argued > > >> > > >> Regards > > >> > > >> Peppe > > >> > > >> > > >> > > >> > > >> --- Mer 17/12/08, Rahkonen Jukka > <jukka.rahko...@mmmtike.fi> ha > > scritto: > > >> > > >> > Da: Rahkonen Jukka > <jukka.rahko...@mmmtike.fi> > > >> > Oggetto: Re: [JPP-Devel] R: R: > IndexOutOfBoundsExceptions with Java > > >> > 1.6.0_11 > > >> > A: "OpenJump develop and use" > <jump-pilot-devel@lists.sourceforge.net > > >, > > >> > "OpenJump develop and use" > <jump-pilot-devel@lists.sourceforge.net> > > >> > Data: Mercoledì 17 dicembre 2008, 20:25 > > >> > Just right, Larry. I have been > digitising for half an hour > > >> > now trying to catch the error but with > no luck. I would say > > >> > this is not any serious bug. Even when > it occurs no work > > >> > gets lost. > > >> > > > >> > But hey! Right now I know how to reach > the error. > > >> > - select either draw polygon or draw > linestring tool > > >> > (basic, not the constrained ones) > > >> > - RIGHT click on map and get the menu > > >> > - LEFT click on map, and there it is! > > >> > > > >> > Perhaps this has nothing to do with Java > update and happens > > >> > only when doing real work in a hurry but > have a try with > > >> > other Java versions. > > >> > > > >> > -Jukka- > > >> > > > >> > > > >> > -----Alkuperäinen viesti----- > > >> > Lähettäjä: Larry Becker > [mailto:becker.la...@gmail.com] > > >> > Lähetetty: ke 17.12.2008 20:52 > > >> > Vastaanottaja: OpenJump develop and use > > >> > Aihe: Re: [JPP-Devel] R: R: > IndexOutOfBoundsExceptions with > > >> > Java 1.6.0_11 > > >> > > > >> > Yes, and I doubt that even Jukka can > reliably reproduce the > > >> > problem. I've > > >> > actually gotten array out of bounds > exceptions myself using > > >> > SkyJUMP, but it > > >> > was always when I was doing actual GIS > work and I > > >> > didn't take the time to > > >> > record the exception. It never seems to > happen during > > >> > testing. :-( > > >> > > > >> > Larry > > >> > > > >> > On Wed, Dec 17, 2008 at 12:01 PM, Stefan > Steiniger > > >> > <sst...@geo.uzh.ch>wrote: > > >> > > > >> > > I think you need to give Jukka the > code, not Paolo :) > > >> > > > > >> > > stefan > > >> > > > > >> > > Sunburned Surveyor wrote: > > >> > > > This is a good point Larry. > > >> > > > > > >> > > > What I could do is give Paolo > a build to try out. > > >> > Then at least we can > > >> > > > know if we've eliminated > the problem for him. > > >> > > > > > >> > > > It is also possible that the > existing code lacks > > >> > some robustness. I > > >> > > > see no reason why the getShape > method of this > > >> > particular cursor tool > > >> > > > should be called with an empty > list of > > >> > coordinates. So perhaps we > > >> > > > could improve the error > checking here. At a > > >> > minimum, it may be better > > >> > > > to throw an > IllegalStateException with an error > > >> > message specific to > > >> > > > the problem at hand which > wraps the exception > > >> > coming from the > > >> > > > ArrayList. > > >> > > > > > >> > > > This is probably something > that could be > > >> > addressed in the parent > > >> > > > class, the MulitClickTool. > > >> > > > > > >> > > > But I shouldn't say > anymore until I look > > >> > closer at the code, or I am > > >> > > > liable to make an arse out of > myself. > > >> > > > > > >> > > > Thanks again for your input. > > >> > > > > > >> > > > Landon > > >> > > > > >> > > > > >> > > > > >> > > > >> > > > > ------------------------------------------------------------------------------ > > >> > > SF.Net email is Sponsored by MIX09, > March 18-20, 2009 > > >> > in Las Vegas, Nevada. > > >> > > The future of the web can't > happen without you. > > >> > Join us at MIX09 to help > > >> > > pave the way to the Next Web now. > Learn more and > > >> > register at > > >> > > > > >> > > > > >> > > > >> > > > > http://ad.doubleclick.net/clk;208669438;13503038;i?http://2009.visitmix.com/ > > >> > > > _______________________________________________ > > >> > > Jump-pilot-devel mailing list > > >> > > > Jump-pilot-devel@lists.sourceforge.net > > >> > > > > >> > > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > > >> > > > > >> > > > >> > > > >> > > > >> > -- > > >> > http://amusingprogrammer.blogspot.com/ > > >> > > > >> > > > >> > > > >> > > > >> > > > > ------------------------------------------------------------------------------ > > >> > SF.Net email is Sponsored by MIX09, > March 18-20, 2009 in > > >> > Las Vegas, Nevada. > > >> > The future of the web can't happen > without you. Join > > >> > us at MIX09 to help > > >> > pave the way to the Next Web now. Learn > more and register > > >> > at > > >> > > > >> > > > > http://ad.doubleclick.net/clk;208669438;13503038;i?http://2009.visitmix.com/ > > >> > > _______________________________________________ > > >> > Jump-pilot-devel mailing list > > >> > Jump-pilot-devel@lists.sourceforge.net > > >> > > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > > >> > > >> > > >> > > >> > > >> > > >> > > > ------------------------------------------------------------------------------ > > >> SF.Net email is Sponsored by MIX09, March > 18-20, 2009 in Las Vegas, > > >> Nevada. > > >> The future of the web can't happen > without you. Join us at MIX09 to > > help > > >> pave the way to the Next Web now. Learn more > and register at > > >> > > >> > > > http://ad.doubleclick.net/clk;208669438;13503038;i?http://2009.visitmix.com/ > > >> > _______________________________________________ > > >> Jump-pilot-devel mailing list > > >> Jump-pilot-devel@lists.sourceforge.net > > >> > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > > > > > > > > > > > > -- > > > http://amusingprogrammer.blogspot.com/ > > > > > > > > > ------------------------------------------------------------------------------ > > > SF.Net email is Sponsored by MIX09, March 18-20, > 2009 in Las Vegas, > > Nevada. > > > The future of the web can't happen without > you. Join us at MIX09 to help > > > pave the way to the Next Web now. Learn more and > register at > > > > > > http://ad.doubleclick.net/clk;208669438;13503038;i?http://2009.visitmix.com/ > > > _______________________________________________ > > > Jump-pilot-devel mailing list > > > Jump-pilot-devel@lists.sourceforge.net > > > > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > > > > > > > > > > > > > ------------------------------------------------------------------------------ > > SF.Net email is Sponsored by MIX09, March 18-20, 2009 > in Las Vegas, Nevada. > > The future of the web can't happen without you. > Join us at MIX09 to help > > pave the way to the Next Web now. Learn more and > register at > > > > > http://ad.doubleclick.net/clk;208669438;13503038;i?http://2009.visitmix.com/ > > _______________________________________________ > > Jump-pilot-devel mailing list > > Jump-pilot-devel@lists.sourceforge.net > > > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > > > > > > -- > http://amusingprogrammer.blogspot.com/ > ------------------------------------------------------------------------------ > SF.Net email is Sponsored by MIX09, March 18-20, 2009 in > Las Vegas, Nevada. > The future of the web can't happen without you. Join > us at MIX09 to help > pave the way to the Next Web now. Learn more and register > at > http://ad.doubleclick.net/clk;208669438;13503038;i?http://2009.visitmix.com/_______________________________________________ > Jump-pilot-devel mailing list > Jump-pilot-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel ------------------------------------------------------------------------------ SF.Net email is Sponsored by MIX09, March 18-20, 2009 in Las Vegas, Nevada. The future of the web can't happen without you. Join us at MIX09 to help pave the way to the Next Web now. Learn more and register at http://ad.doubleclick.net/clk;208669438;13503038;i?http://2009.visitmix.com/ _______________________________________________ Jump-pilot-devel mailing list Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
