Martin Desruisseaux ha scritto: > Andrea Aime a écrit : >> Brent Owens ha scritto: >>> I thought this was already discussed and voted down? Actual changes >>> before this event will be lost in the formatting changes, and that >>> was the main reason for no auto-formatting. >> >> What? No, exactly the opposite. > > I can understand both point of views. It could also be argued that the > formatting problem would not occurs if the jaloppy formatting were not > applied automatically.
If no automated formatting was done, you mean. Too bad many developers grew used to auto-format with their IDE before saving. I do, I never save before fixing imports and reformat, it has become part of my "save file" keystroke sequence. > But I realize that this argument may be more > difficult to hold as the number of developpers in the projet increase. > > I agree about using a uniform coding convention, and my understanding is > that we agreed a while ago on Sun's coding conventions (is that right?). > If some parts of my code violate the Sun's coding convention, those > violations need to be fixed. However, last time I tried jaloppy, I found > that the damages (in term of making code harder to read, like breaking > alignments) was wider than the fixes. The problem is, alignements are not part of the convention, and are hard to maintain... > > In the specific case of the coverage branch merge, I just asked (and > obtained) 8 vacation days today. I'm taking vacation (for development > work actually) from June 21 to July 2. Note however that I will have to > share my time between the coverage branch merge and JSR-275 (javax.units > specification), which is in urgent need for care if we don't want Sun to > withdrawn it like the old JSR-108 (by the way, if anyone wish to comment > on the very early and partial draft posted on > https://jsr-275.dev.java.net/servlets/ProjectDocumentList?folderID=5344 > , that would be appreciated). I hope you realize that every file that has been modified in the gc branch has _already_ been formatted with Jalopy. Simone just tried to apply our developer's guide... Cheers Andrea _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
