Cory Horner ha scritto: > Andrea Aime wrote: > >> If you want to try out jalopy with the current settings on a module, >> go into it and type: >> mvn -Pjalopy jalopy:format >> >> The configuration file is located in the maven/build-configs module >> (so that it can be shared). Note that I disabled jalopy javadoc >> comment reformat because it simply breaks on most of our javadoc >> comments due to a fragile parser combined with our numerous HTML >> errors in the comments (forbidden characters, unclosed tags and the >> like). >> >> Let me know what you think of it. >> > This is a thing of beauty! I tried it out on ext/brewer (to check that I > got all my changes in from the 2.2.x branch) and it worked quite well. > > I've merged your changes onto the 2.2.x branch, so we can reformat > modules there as well, should we choose to. > > Hopefully as releases approach and branches disappear, we'll have an > opportunity to run this on all the code. Sure it will make diffs > difficult across this point, but it must be done sometime... and > preferably sooner than later.
Well, my plan was to suggest module manintainer to enable the profile on their modules, if they wish, as soon as the grid coverage branch merge is complete. I do believe that for the FM branch doing a full reformat of both trunk and FM will ease merging. Cheers Andrea ------------------------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
