freenet/fred-staging — pull request #234 by Abominatio (2014-05-31 21:16:20+00:00)
Title: Coding Standards Audit

I’ve been lurking for a while, just following development. Had a bit of free time lately, so I made the decision to contribute. I noticed after years of many people contributing, the code-base has gotten a little messy. So…I give you 110 commits to fix just that! Yes a lot of the code was reformatted through automation, but a lot was done manually as well. I also built and tested before submitting this to make sure I didn’t just break everything.

All of the changes accomplish the following:
[] Verifies all code meets the “Freenet Java Coding Standards”. - (https://wiki.freenetproject.org/Coding_standards)
[] Wraps lines at 120 characters (Yes I know the standard says 100, but this helped as I didn’t need to wrap as many lines).
[] Organizes imports into groups of ‘JDK’ and ‘non-JDK’ imports.
[] Reformats loads of messy comments.
[] Adds what I believe to be the standard license statement to most of the files. (I used the most commonly used license statement. If any sort of license existed previously, I left it alone. PLEASE let me know if this is the correct one or if this needs to change/removed. NOT trying to fuck up your licensing).
[] Creates proper code blocks where none existed. This means no more ‘If’ statements without opening and closing curly brackets!

I tried a few things to keep the diff noise to a minimum, but sadly I wasn’t able to do so. Each commit is broken down into no more than 10 files changed at a time.

I sincerely hope this is helpful and will be glad to fix any problems you guys spot in my changes.