You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@accumulo.apache.org by Christopher <ct...@apache.org> on 2013/10/29 00:56:48 UTC

Whitespace

Hey Vines,

Your last commit modified a bunch of blank lines to indent them,
resulting in trailing whitespace. After we switched to git, we updated
the formatter to strip those. Not all code has been reformatted yet,
but for some of the code that has been (because there have been edits
to it), it'd be nice if it wasn't reverted to have trailing whitespace
again.

Please update your Eclipse formatter to the latest in master/contrib.

Thanks.

--
Christopher L Tubbs II
http://gravatar.com/ctubbsii

Re: Whitespace

Posted by John Vines <vi...@apache.org>.

My mistake, I'll fix that first thing tomorrow

Sent from my phone, please pardon the typos and brevity.
On Oct 28, 2013 7:56 PM, "Christopher" <ct...@apache.org> wrote:

> Hey Vines,
>
> Your last commit modified a bunch of blank lines to indent them,
> resulting in trailing whitespace. After we switched to git, we updated
> the formatter to strip those. Not all code has been reformatted yet,
> but for some of the code that has been (because there have been edits
> to it), it'd be nice if it wasn't reverted to have trailing whitespace
> again.
>
> Please update your Eclipse formatter to the latest in master/contrib.
>
> Thanks.
>
> --
> Christopher L Tubbs II
> http://gravatar.com/ctubbsii
>