@@ -4,3 +4,4 @@ target
 *.ipr
 *.iws
 *.idea
+.checkstyle
@@ -2,6 +2,7 @@ eclipse.preferences.version=1
 org.eclipse.jdt.core.compiler.codegen.targetPlatform=1.7
 org.eclipse.jdt.core.compiler.compliance=1.7
 org.eclipse.jdt.core.compiler.problem.forbiddenReference=warning
+org.eclipse.jdt.core.compiler.processAnnotations=disabled
 org.eclipse.jdt.core.compiler.source=1.7
 org.eclipse.jdt.core.formatter.align_type_members_on_columns=false
 org.eclipse.jdt.core.formatter.alignment_for_arguments_in_allocation_expression=16
 
@@ -29,6 +29,9 @@
 	<description>The API jar containing the annotations and the runtime helpers</description>
 	<name>AndroidAnnotations API</name>
 
+    <properties>
+        <main.basedir>${project.parent.basedir}</main.basedir>
+    </properties>
 
 	<dependencies>
 		<dependency>
 
@@ -29,7 +29,7 @@
 import android.os.Looper;
 import android.util.Log;
 
-public class BackgroundExecutor {
+public final class BackgroundExecutor {
 
 	private static final String TAG = "BackgroundExecutor";
 
@@ -82,12 +82,16 @@ private BackgroundExecutor() {
 	 *            the task to execute
 	 * @param delay
 	 *            the time from now to delay execution, in milliseconds
-	 * @throws IllegalArgumentException
-	 *             if <code>delay</code> is strictly positive and the current
-	 *             executor does not support scheduling (if
-	 *             {@link #setExecutor(Executor)} has been called with such an
-	 *             executor)
+	 * 
+	 *            if <code>delay</code> is strictly positive and the current
+	 *            executor does not support scheduling (if
+	 *            {@link #setExecutor(Executor)} has been called with such an
+	 *            executor)
 	 * @return Future associated to the running task
+	 * 
+	 * @throws IllegalArgumentException
+	 *             if the current executor set by {@link #setExecutor(Executor)}
+	 *             does not support scheduling
 	 */
 	private static Future<?> directExecute(Runnable runnable, int delay) {
 		Future<?> future = null;
 
@@ -27,7 +27,7 @@
  * This class provide operations for
  * {@link org.androidannotations.annotations.UiThread UiThread} tasks.
  */
-public class UiThreadExecutor {
+public final class UiThreadExecutor {
 
 	private static final Handler HANDLER = new Handler(Looper.getMainLooper()) {
 		@Override
@@ -112,7 +112,7 @@ public static void cancelAll(String id) {
 		HANDLER.removeCallbacksAndMessages(token);
 	}
 
-	private static class Token {
+	private static final class Token {
 		int runnablesCount = 0;
 		final String id;
 
 
@@ -111,6 +111,7 @@
  * }
  * </pre>
  */
+@SuppressWarnings("checkstyle:finalclass")
 public class ViewServer implements Runnable {
 	/**
 	 * The default port used to start view servers.
@@ -518,7 +519,7 @@ public void write(int oneByte) throws IOException {
 		}
 	}
 
-	private static class NoopViewServer extends ViewServer {
+	private static final class NoopViewServer extends ViewServer {
 		private NoopViewServer() {
 		}
 
 
@@ -23,7 +23,7 @@
 /**
  * Utility class for working with {@link Bundle} objects.
  */
-public class BundleHelper {
+public final class BundleHelper {
 
 	private BundleHelper() {
 
 
@@ -22,7 +22,7 @@
  * Reflection utils to call methods which are accessible in RoboGuice package
  * but not from outside.
  */
-public class RoboGuiceHelper {
+public final class RoboGuiceHelper {
 
 	private RoboGuiceHelper() {
 	}
 
@@ -61,7 +61,7 @@ public static String serialize(Set<String> set) {
 					.endDocument();
 
 		} catch (IllegalArgumentException | IOException | IllegalStateException e) {
-
+			// should never happen
 		}
 
 		return writer.toString();
 
@@ -30,6 +30,9 @@
 	<name>AndroidAnnotations Bundle</name>
 	<packaging>pom</packaging>
 
+    <properties>
+        <main.basedir>${project.parent.basedir}</main.basedir>
+    </properties>
 
 	<dependencies>
 		<dependency>
-Original file line number
+Diff line change
 *.ipr
 *.iws
 *.idea
 +.checkstyle
Original file line number	Diff line number	Diff line change
`@@ -111,6 +111,7 @@`
`111`	`111`	`* }`
`112`	`112`	`* </pre>`
`113`	`113`	`*/`
	`114`	`+@SuppressWarnings("checkstyle:finalclass")`
`114`	`115`	`public class ViewServer implements Runnable {`
`115`	`116`	`/**`
`116`	`117`	`* The default port used to start view servers.`
`@@ -518,7 +519,7 @@ public void write(int oneByte) throws IOException {`
`518`	`519`	`}`
`519`	`520`	`}`
`520`	`521`
`521`		`- private static class NoopViewServer extends ViewServer {`
	`522`	`+ private static final class NoopViewServer extends ViewServer {`
`522`	`523`	`private NoopViewServer() {`
`523`	`524`	`}`
`524`	`525`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@`
`22`	`22`	`* Reflection utils to call methods which are accessible in RoboGuice package`
`23`	`23`	`* but not from outside.`
`24`	`24`	`*/`
`25`		`-public class RoboGuiceHelper {`
	`25`	`+public final class RoboGuiceHelper {`
`26`	`26`
`27`	`27`	`private RoboGuiceHelper() {`
`28`	`28`	`}`
Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,7 @@ public static String serialize(Set<String> set) {`
`61`	`61`	`.endDocument();`
`62`	`62`
`63`	`63`	`} catch (IllegalArgumentException \| IOException \| IllegalStateException e) {`
`64`		`-`
	`64`	`+ // should never happen`
`65`	`65`	`}`
`66`	`66`
`67`	`67`	`return writer.toString();`