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: us...@infra.apache.org With regards, Apache Git Services
