It's just that I don't have a lot of time right now to try and get IDEA or Jalopy to match our CheckStyle setup. If CheckStyle is going to be held up as the arbitrator of coding style, it would be nice if a code reformatter could reach the CheckStyle configuration directly.
If anyone would like to volunteer to try and match IDEA's configuration with our CheckStyle file, that would be very helpful. I made some changes against the IDEA defaults, but apparently they are not quite right. I'd like to reduce the coding inconsistencies, but I don't want to introduce a lot of manual changes right before a release. The changes from the IDEA norm I've identified so far are documented in my comment of 12/31. http://issues.apache.org/bugzilla/show_bug.cgi?id=36306 Or, alternatively, we could develop a CheckStyle file to match the IDEA norm, and use that instead. I think the important thing is to have a coding style that we can enforce with an automatic tool, so as to not waste volunteer time on mechanical tasks. -Ted. On 1/17/06, Frank W. Zammetti <[EMAIL PROTECTED]> wrote: > You may want to have a look at Jalopy: > > jalopy.sf.net > > There are plugins for various IDEs and editors, as well as an Ant task > (IIRC, you can use Ant tasks from Maven?). It won't fix everything, but > it will help enforce many of the more common coding standards. It > should be possible to configure its rules to match those of CheckStyle, > so you could then use CheckStyle to manually deal with what remains that > Jalopy doesn't fix. > > Frank > > Don Brown wrote: > > It just fills the errors list with checkstyle errors, that you can then > > click on to be taken to the file and line. It was very handy for > > cleaning up Scripting. > > > > Don > > > > Ted Husted wrote: > > > >> Does it actually reformat the code? Or just report on what should be > >> reformatted? > >> > >> On 1/17/06, Don Brown <[EMAIL PROTECTED]> wrote: > >> > >> > >>> I've been using jEdit's checkstyle plugin. I had to temporary modify > >>> the checkstyle xml to remove those dynamic properties, but otherwise, it > >>> worked great. > >>> > >>> Don > >>> > >>> Ted Husted wrote: > >>> > >>> > >>>> Is anyone using an automatic code reformatting tool that can read a > >>>> CheckStyle configuration? > >>>> > >>>> I'd like to clean up some of the stylistic errors, but I would prefer > >>>> that most of the reformatting be done by a proven tool. I have utter > >>>> confidence in IDEA, but I don't know if we can perfectly align the > >>>> settings with CheckStyle. > >>>> > >>>> For more, see > >>>> > >>>> * http://issues.apache.org/bugzilla/show_bug.cgi?id=36306 > >>>> > >>>> Having CheckStyle standards is great, but I would rather have a > >>>> consistent code style that we can enforce with an automatic > >>>> reformatting tool. > >>>> > >>>> -Ted. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]