+1 to change the formatter.
On Wed, Mar 6, 2013 at 3:18 PM, Billie Rinaldi <[email protected]> wrote: > I personally would prefer if we changed our formatter to have no spaces on > blank lines. git calls these "whitespace errors" when you're applying a > patch. > > Billie > > > On Wed, Mar 6, 2013 at 11:57 AM, Christopher <[email protected]> wrote: > >> Wouldn't that rule conflict with our formatter? I'm pretty sure our >> formatter allows "blank" lines to be indented with spaces. >> >> -- >> Christopher L Tubbs II >> http://gravatar.com/ctubbsii >> >> >> On Wed, Mar 6, 2013 at 10:23 AM, David Medinets >> <[email protected]> wrote: >> > I have a free day due to snowfall and while this is a fairly silly >> > rule, writing a short script to rule all the java files through sed >> > should be fairly painless. As part of this change, I will commit a >> > one-rule checkstyle.xml file which just runs this 'no spaces at end of >> > line' rule. Over time, more rules can be added to that align with the >> > Accumulo community's style guidelines. >> > >> > Any objection? >>
