From ad1ab52a9ed957f97400b0a2837fe7da6a0d7546 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9gis=20D=C3=A9camps?= <regisd@google.com>
Date: Sat, 4 Nov 2017 00:11:10 +0100
Subject: [PATCH] Add google error-prone in compile step. (#223)

* Add google error-prone in compile step.
  Note that FallThrough violations have been reduced to warnings. See issue #222
* Enable error-prone in the java 8 profile only, because the plugin was compiled for java 8, and cannot be used in openjdk7.
---
 pom.xml | 30 +++++++++++++++++++++++++++++-
 1 file changed, 29 insertions(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 2ac7840ff..10f62874f 100644
--- a/pom.xml
+++ b/pom.xml
@@ -235,13 +235,41 @@
   </build>
   <profiles>
     <profile>
-      <!-- Profile for Java 8: Disable Doclint -->
+      <!-- Profile for Java 8: Enable error-prone, Disable Doclint -->
       <id>java8</id>
       <activation>
         <jdk>[1.8,)</jdk>
       </activation>
       <build>
         <plugins>
+          <plugin>
+            <groupId>org.apache.maven.plugins</groupId>
+            <artifactId>maven-compiler-plugin</artifactId>
+            <version>3.3</version>
+            <configuration>
+              <compilerId>javac-with-errorprone</compilerId>
+              <forceJavacCompilerUse>true</forceJavacCompilerUse>
+              <compilerArgs>
+                <arg>-Xep:FallThrough:WARN</arg>
+              </compilerArgs>
+              <source>7</source>
+              <target>7</target>
+            </configuration>
+            <dependencies>
+              <dependency>
+                <groupId>org.codehaus.plexus</groupId>
+                <artifactId>plexus-compiler-javac-errorprone</artifactId>
+                <version>2.8</version>
+              </dependency>
+              <!-- override plexus-compiler-javac-errorprone's dependency on
+                   Error Prone with the latest version -->
+              <dependency>
+                <groupId>com.google.errorprone</groupId>
+                <artifactId>error_prone_core</artifactId>
+                <version>2.1.1</version>
+              </dependency>
+            </dependencies>
+          </plugin>
           <plugin>
             <groupId>org.apache.maven.plugins</groupId>
             <artifactId>maven-javadoc-plugin</artifactId>