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. 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?). 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
