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/

Reply via email to