On 5/31/07, Xavier Hanin <[EMAIL PROTECTED]> wrote:
Hi, We discussed recently about using the eclipse tool to clean up the code to decrease our number of checkstyle errors/warnings. Gilles now you have committed your changes, could we agree on a day where we could make sure we have no pending changes so that one of us can run the tool and commit the cleaned up code without needing a merge? Before we should apply the patch from Stephane: https://issues.apache.org/jira/browse/IVY-518 On my side I have no pending changes, so any day is ok. I'm ok to apply the patch and run the clean up tool, I will have time to do it next week. If anybody else prefer to do it, I'd be happy to oblige :-) We also have to agree on maximum line length, is 100 (the current setting of checkstyle, borrowed from Ant) ok for everybody?
One more thing: I think it will be a good opportunity to fix our eol inconsistencies: using fixcrlf + applying svn:eol-style property. Committers, maybe we should use the Apache settings for svn, to auto apply eol-style: http://www.apache.org/dev/svn-eol-style.txt Xavier
Xavier On 5/24/07, Xavier Hanin <[EMAIL PROTECTED]> wrote: > > On 5/24/07, Gilles Scokart <[EMAIL PROTECTED]> wrote: > > > > Yes, I have pending changes about the buildlist. I'm waiting my > > userID to commit them. > > > > Do you think eclipse formater will really reduce the number of errors > > reported by CheckStyle? If yes, it might be a good idea, but I'm not > > sure it will. > > > Eclipse can fix a lot of code style errors, like import rules, > indentation, spaces, line length and so on. So I think it would reduce the > number of errors significantly and easily. > > I just have one remarks with the eclipse formating using Sun standard > > : the default width of 80 characters is too small. A bigger value > > should be used (120 maybe?). > > > I agree 80 is too small. The checkstyle rules Maarten has added set it > to 100, I think it's the same as for ant (I think the rules we use for the > moment are the same as for ant). Personally I use a wide screen, so long > lines are not a big issue for me, but I understand they make lines less > readable and more difficult to debug. So maybe 100 is a good compromise. > > Opinions? > > Xavier > > Gilles > > > > > > 2007/5/24, Xavier Hanin < [EMAIL PROTECTED]>: > > > I don't see the patch either. > > > > > > I've created a JIRA issue for this kind of patch: > > > https://issues.apache.org/jira/browse/IVY-515 > > > > > > This will avoid to create one issue for each patch, which doesn't > > really > > > make sense. > > > > > > BTW, for the coding style problems, maybe we could use a tool to > > make things > > > cleaner. Eclipse has a pretty good tool to clean code and make it > > closer to > > > coding conventions. Running it is pretty simple, and since we agreed > > on the > > > standard coding convention, we don't even to configure it. The main > > problem > > > is to synchronize among developers to know who and when this will be > > done, > > > because it may result in a lot of conflict for subsequent merges. > > This also > > > mean that patch currently in JIRA will be more difficult to apply. > > But this > > > is the only way I see to reduce significantly the number of > > checkstyle > > > errors (this won't fix the _ prefix, but it will fix a lot of more > > trivial > > > errors). > > > > > > So, what do you think? Should we use this tool? If so, when? Do you > > have > > > pending changes in your code base? > > > > > > Xavier > > > > > > On 5/24/07, Gilles Scokart <[EMAIL PROTECTED]> wrote: > > > > > > > > Sorry, I didn't see your patch. Was it suposed to be attached to > > the > > > > mail? > > > > > > > > You could maybe add a Jira issue for it. > > > > > > > > Thanks, > > > > > > > > Gilles > > > > > > > > 2007/5/24, Kevin Jackson < [EMAIL PROTECTED]>: > > > > > Hi all, > > > > > > > > > > Here are some javadoc and style changes to make Ivy conform > > (slightly) > > > > > more with the Apache style - hopefully kill off some checkstyle > > > > > warnings :) > > > > > > > > > > Kev > > > > > > > > > > > > > > > > > -- > > > > Gilles SCOKART > > > > > > > > > > > > > > > > -- > > > Xavier Hanin - Independent Java Consultant > > > Manage your dependencies with Ivy! > > > http://incubator.apache.org/ivy/ > > > > > > > > > -- > > Gilles SCOKART > > > > > > -- > Xavier Hanin - Independent Java Consultant > Manage your dependencies with Ivy! > http://incubator.apache.org/ivy/ > -- Xavier Hanin - Independent Java Consultant Manage your dependencies with Ivy! http://incubator.apache.org/ivy/
-- Xavier Hanin - Independent Java Consultant Manage your dependencies with Ivy! http://incubator.apache.org/ivy/
