In general I am pretty -1 on any major reformatting like this as whenever I
have seen it done it does more harm than good. Maybe on a few classes here
and there perhaps that are known to be a mess.
We should really be on people when they commit stuff to use proper
formatting. As for patches on misformatted classes I just think one has to
deal with it and avoid the temptation to reformat.
Also related I put together some patch submission guideline for geoserver a
while back that mention formatting.
http://docs.geoserver.org/latest/en/developer/policies/patches.html
$0.02
On Friday, August 31, 2012, Jody Garnett wrote:
> Maybe reformatting trunk right before the new stable braches off then?
>
> Yes that is what I was trying to say.
>
> That way both master and the new branch start from the same (formatted)
> place.
>
> Jody
>
>
--
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
