accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] ctubbsii commented on issue #496: Update formatter codestyle
Date Wed, 06 Jun 2018 23:22:49 GMT
ctubbsii commented on issue #496: Update formatter codestyle
URL: https://github.com/apache/accumulo/issues/496#issuecomment-395244430
 
 
   @alerman This issue isn't about the 100 character wrap. That was previously discussed and
agreed upon, because it's easier on folks with poor eyesight who need to increase their font
size, is a de facto standard by many projects (such as those using Google's Style Guide),
and allows for better side-by-side diffs on GitHub and in other diff viewers. Since that was
a recent change, it is not likely to be revisited any time soon.
   
   Instead, this issue is more about updating the additional options that have been added
to the formatter since Java 8, such as where to wrap lambdas, when to wrap params, and how
to wrap array literals, etc. For many of these, we are just using the defaults, because they
did not exist last time we examined our style configuration.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message