We should stay on the list .... and that stack trace should go into the bug report :-)
So now we are making some progress it is ParameterGroup that is having trouble? Those classes are not really supposed to be that mutable (and thus not need cloning). Oh wait maybe it was ParameterGroupDescriptor that was not mutable ... Can put a break point in your debugger and see what actual instance of ParameterGroup is giving you trouble; and what the values of the parameter group are? The CRS code does its best to reuse and recycle objects (and to make object immutable) so I am a bit shocked to see clone being used... Jody On 13/05/2010, at 5:25 AM, Joachim Van der Auwera wrote: > Dear Jody, > > Thanks for the effort, and strangely enough, in my tests (which included many > other modules than referencing), I got the following stacktrace > > java.lang.AssertionError: java.lang.CloneNotSupportedException: > org.geotools.parameter.ParameterGroup > at > org.geotools.parameter.AbstractParameter.clone(AbstractParameter.java:164) > at org.geotools.parameter.ParameterGroup.clone(ParameterGroup.java:439) > at org.geotools.parameter.ParameterGroup.clone(ParameterGroup.java:63) > at > org.geotools.referencing.operation.DefiningConversion.<init>(DefiningConversion.java:93) > at > org.geotools.referencing.operation.DefiningConversion.<init>(DefiningConversion.java:66) > at org.geotools.referencing.wkt.Parser.parseProjCS(Parser.java:909) > at > org.geotools.referencing.wkt.Parser.parseCoordinateReferenceSystem(Parser.java:222) > at > org.geotools.referencing.wkt.Parser.parseCoordinateReferenceSystem(Parser.java:201) > at > org.geotools.referencing.factory.ReferencingObjectFactory.createFromWKT(ReferencingObjectFactory.java:1088) > at org.geotools.referencing.CRS.parseWKT(CRS.java:450) > > > But the patch indeed works just fine. I am very puzzled. > > Thanks for looking into this. > Kind regards, > Joachim > > > Jody Garnett wrote: >> Comments inline. >> >> On 12/05/2010, at 10:02 PM, Joachim Van der Auwera wrote: >> >> >>> Dear Jody, >>> >>> Nobody on our team has commit rights. >>> >> >> Well that can be fixed; I see andrea has already indicated how :-) We do try >> to be an open welcoming project if there is anything we can do to assist let >> us know. >> >> >>> The patch does not literally include a patch, but it includes details about >>> the fix and how to reproduce the problem (fix just involves adding >>> "implements Cloneable" on "org.geotools.referencing.wkt.Formattable"). >>> >> >> Yeah; I just tried it now - see the >> http://jira.codehaus.org/browse/GEOT-3062 for details; and a patch you can >> apply to your own checkout. >> The test case runs if Formatter implements Clonable or not; so more work is >> needed the bug report is not complete. Give it a go; perhaps I have >> implemented the test case wrong? >> >> Jody >> >> >>> Kind regards, >>> Joachim >>> >>> On 05/12/2010 08:58 AM, Jody Garnett wrote: >>> >>>> Sure; do you have someone with commit access to apply the patch? Is there >>>> a patch :-) Is there a test case... >>>> >>>> I am not going to call "out of the pool" until Friday according to the >>>> CSIRO crew doing the geoserver release; so you have some time to get fixes >>>> in. >>>> >>>> Jody >>>> >>>> On 12/05/2010, at 4:17 PM, Joachim Van der Auwera wrote: >>>> >>>> >>>>> Dear Jody, >>>>> >>>>> Would it still be possible to consider GEOT-3062 for inclusion in this >>>>> release? >>>>> That would help Geomajas to use the latest version. >>>>> >>>>> Thanks, >>>>> Joachim >>>>> >>>>> On 05/11/2010 09:19 AM, Jody Garnett wrote: >>>>> >>>>>> We are going to get a GeoTools 2.6.4 out this week :-) >>>>>> >>>>>> By happy chance both uDig and GeoServer are getting released at the same >>>>>> time; so this release of GeoTools will be subject to lots of testing. >>>>>> >>>>>> Michael can your interpolate function be brought in from the cold? I >>>>>> think it has gotten some review now ... >>>>>> >>>>>> Is there any other work waiting in the wings? My plan is to tag on >>>>>> Friday; allow GeoServer to go through Cite testing and if all goes well >>>>>> uploaded to SourceForge on Monday. >>>>>> >>>>>> What can be done to help? >>>>>> - answer this email if you have pending work (either to get in; or to be >>>>>> delayed until next week) >>>>>> - testing the project zip when available >>>>>> - go through Jira and look for issues that are already fixed >>>>>> - blog posts about functionality added during the 2.6.4 release window >>>>>> - testing! >>>>>> >>>>>> Jody >>>>>> ------------------------------------------------------------------------------ >>>>>> >>>>>> __ >>>>>> >>> -- >>> Joachim Van der Auwera >>> Geosparc, supporting the Geomajas GIS community >>> >>> >> >> > > > -- > Joachim Van der Auwera > PROGS bvba, progs.be > ------------------------------------------------------------------------------ _______________________________________________ Geotools-devel mailing list Geotools-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
