Refactorings pmd and intellij rules

[jeduardocosta]

refactorings based on pmd rules:

• https://github.com/feroult/yawp/commit/552412d1668f5b18c1c9fee4a0b2e2d9e0eca435 http://pmd.sourceforge.net/pmd-5.3.2/pmd-java/rules/java/unnecessary.html#UnnecessaryReturn
• https://github.com/feroult/yawp/commit/354a68761c64335915bec2ca247dad7b4c530fc4 http://pmd.sourceforge.net/pmd-5.3.2/pmd-java/rules/java/design.html#FieldDeclarationsShouldBeAtStartOfClass

refactorings based on intellij code analysis the following items were removed:

• https://github.com/feroult/yawp/commit/3681abd13687f74f209ef8e5667968ed83845943 redundant modifier for interface methods
• https://github.com/feroult/yawp/commit/a6fe3688decf3a86a8cd2de368c8ee933bc1f091 unused imports
• https://github.com/feroult/yawp/commit/d6f079f5de31f080cc26a52e9d302fa36c16d259 explicit argument types (added diamond operator - jdk 1.7+)

[coveralls]

Coverage decreased (-0.07%) to 63.552% when pulling d6f079f5de31f080cc26a52e9d302fa36c16d259 on jeduardocosta:refactorings-pmd-and-intellij-rules into 652a344dad86d3d70be4d041889deb8379eed2f8 on feroult:master.

[feroult]

Thanks @jeduardocosta!