Sunburned Surveyor ha scritto: > I did a very dangerous thing today and downloaded the source code for > GeoTools. I got the source code for the shapefile module loaded into > Eclipse and set-up the libraries it required. I thought this was the > best way to avoid asking stupid questions Joday and Andrea have to > answer. > > At any rate, I started poking around in the DbaseFileHeader class > looking for some answers I couldn't find in the Javadoc. I had > originally planned on sending Andrea just a couple of patches, but I > think there may be more opportunities to improve the code > comments/functionality. I was wondering if I could get a little space > in the SVN to do some work on the code. This would be a fork of the > Shapefile module obviously, and I would get approval from Andrea and > others before any changes were commited to the trunk.
I think you could use the spike area? So that we don't add another minute to the build... As for hammering hard changes on the code you're obviously free to do so in your fork, but please when you come back to provide patches make it so that each one does one change and one change only, otherwise reviewing them will be too hard. My personal preference is that you get down to business and be productive first, and if it's possible without hindering too much your work, to get a stream of smaller patches to be applied a bit at a time so that it's easier to pinpoint the cause in case a change makes things go south in an application using GT2 (either uDig or GeoServer, hopefully OpenJump too in the future). Cheers Andrea -- Andrea Aime OpenGeo - http://opengeo.org Expert service straight from the developers. ------------------------------------------------------------------------------ Come build with us! The BlackBerry® Developer Conference in SF, CA is the only developer event you need to attend this year. Jumpstart your developing skills, take BlackBerry mobile applications to market and stay ahead of the curve. Join us from November 9-12, 2009. Register now! http://p.sf.net/sfu/devconf _______________________________________________ Geotools-devel mailing list Geotools-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
