Il 14/03/2012 16:04, Emmanuel Lécharny ha scritto:
Le 3/14/12 3:46 PM, Francesco Chicchiriccò a écrit :
On 14/03/2012 15:40, Emmanuel Lécharny wrote:
Hi,
I see that this commit contains a *lot* of changes like :
@@ -146,26 +146,21 @@ public class Configuration extends BaseP
// Workflow definition stuff
final WorkflowDefinitionTO workflowDef =
wfRestClient.getDefinition();
- WebMarkupContainer workflowDefContainer =
- new WebMarkupContainer("workflowDefContainer");
+ WebMarkupContainer workflowDefContainer = new
WebMarkupContainer("workflowDefContainer");
which is a clear indication that some of you aren't using the same
indentation policy.
Hi Emmanuel,
this is happening because we used to have 80 columns constraint for
code lines but we moved to 120 columns contextually with Apache
Syncope set up.
Now, as soon as I open a file formatted with former constraint, I
just ask my IDE to reformat and that's the result.
I understand. Format *all* the files in one shot, then commit. That
would be way better than spreading such modifications in many commits.
It *would* be way better, but I'd prefer to check this instead of
relying upon IDE features only.
Anyway, I'll try to group such non-functional modifications as much as I
can.
Regards.
--
Francesco Chicchiriccò
Apache Cocoon PMC and Apache Syncope PPMC Member
http://people.apache.org/~ilgrosso/
