Hi Gerson, Thanks very much for this. Bug fixes and improvements are always welcome !
Given the big workload of the core developers and inevitable delays this leads to in reviewing patches, the best things is for you to go ahead and create an issue in Jira plus an associated pull request citing the issue number. That way it will be on the list for the module maintainer (Justin). Michael On 31 October 2012 15:46, Gerson Galang <[email protected]> wrote: > Hi, > > A month ago, I reported an issue we picked up where the CRS info from the > parsed GML document gets lost in the parsing process. Here's a link to my > posting again which details the issue I found.. > > http://osgeo-org.1560.n6.nabble.com/Missing-CRS-on-the-SimpleFeature-objects-td5005105.html > > After digging into the source code, I found a place where I could insert the > missing CRS. My geotools fork contains the patch which fixes the problem.. > > https://github.com/gerson721/geotools/commit/fd63ce418f459ca31f7a0a42ce7a6678879d3100 > > Is my fix acceptable with you guys? If it is, I'll add a JIRA entry for it > and make a pull request. > > Regards, > Gerson > > > > > ------------------------------------------------------------------------------ > Everyone hates slow websites. So do we. > Make your web apps faster with AppDynamics > Download AppDynamics Lite for free today: > http://p.sf.net/sfu/appdyn_sfd2d_oct > _______________________________________________ > GeoTools-GT2-Users mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/geotools-gt2-users > ------------------------------------------------------------------------------ Everyone hates slow websites. So do we. Make your web apps faster with AppDynamics Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_sfd2d_oct _______________________________________________ GeoTools-GT2-Users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-gt2-users
