diff options
author | Arne H Juul <arnej@yahoo-inc.com> | 2017-04-27 13:20:31 +0200 |
---|---|---|
committer | Arne H Juul <arnej@yahoo-inc.com> | 2017-04-30 18:07:46 +0200 |
commit | 60ed0087656bba260da499d32346c09392ab2758 (patch) | |
tree | 032540fe492afeaf144f875462efe7af14ef3d80 /pom.xml | |
parent | 5ecd96e695b84c6fa6e618772f396b5a46b99f9b (diff) |
add -Werror as default compiler option
lint warnings considered less useful are disabled by default:
* "lint:serial" warns whenever you extend a serializable
class, even for anonymous classes. We don't use java
serialization anyway so turn this off.
* "lint:try" warns when you want to do setup-teardown
style using an AutoClosable, there's no good reason
that I can see why this should produce a warning.
* "lint:processing" warns when annotations aren't
"claimed", but we don't have control over the
various annotation processors used, so we can't fix
these warnings.
* "lint:varargs" gives totally bogus warnings, possibly
because the entire varargs mechanism is badly designed.
It triggers for any varargs method using Arrays.asList()
on its arguments, which most of the convenience methods
really should do.
Diffstat (limited to 'pom.xml')
-rw-r--r-- | pom.xml | 5 |
1 files changed, 5 insertions, 0 deletions
@@ -100,6 +100,11 @@ <showDeprecation>false</showDeprecation> <compilerArgs> <arg>-Xlint:all</arg> + <arg>-Xlint:-serial</arg> + <arg>-Xlint:-try</arg> + <arg>-Xlint:-processing</arg> + <arg>-Xlint:-varargs</arg> + <arg>-Werror</arg> </compilerArgs> </configuration> </plugin> |