On 2/5/06, Wendy Smoak <[EMAIL PROTECTED]> wrote: > The long lines are a problem. Not fixing them is one thing, but it > wants to make this change, which *creates* a line longer than 80 > characters: > > - protected String processorClass = > - "org.apache.struts.chain.ComposableRequestProcessor"; > + protected String processorClass = > "org.apache.struts.chain.ComposableRequestProcessor";
>From the sf.net forum: "Preferences->printer->wrapping->wrap after assignments (make sure this is checked)" I'm not using the GUI, but that was enough of a hint, and r375446 seems to have helped. But Jalopy is still insisting on long lines for method declarations, such as (in ActionForm): public ActionErrors validate(ActionMapping mapping, ServletRequest request) { and for method calls (in ActionRedirect): result.append("parameterString=").append(getParameterString()).append("]"); (though that one could just as well be split into two lines.) It also makes this change: - } else { + } + else { which causes Checkstyle to complain that '} should be on the same line'. Patches for the config file (build/jalopy_struts.xml) are welcome! I'm testing with 'maven jalopy' in struts-action. Since it was already reformatted, it's easy to see (with 'svn diff') where Jalopy is making changes that it shouldn't. Thanks, Wendy --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]