fixed minor code issues (jooby-project#1171)

feffi · jknack · commit 3b552da9615f · 2018-06-25T12:51:49.000-03:00
(cherry picked from commit aac2fdf)
diff --git a/modules/jooby-rxjava/src/main/java/org/jooby/rx/ExecSchedulerHook.java b/modules/jooby-rxjava/src/main/java/org/jooby/rx/ExecSchedulerHook.java
@@ -213,7 +213,7 @@
 
 class ExecSchedulerHook extends RxJavaSchedulersHook {
 
-  private Throwing.Function<String, Scheduler> schedulers;
+  private final Throwing.Function<String, Scheduler> schedulers;
 
   public ExecSchedulerHook(final Map<String, Executor> executors) {
     // we don't want eager initialization of Schedulers
diff --git a/modules/jooby-rxjava/src/main/java/org/jooby/rx/Rx.java b/modules/jooby-rxjava/src/main/java/org/jooby/rx/Rx.java
@@ -357,7 +357,7 @@ static class DeferredSubscriber extends Subscriber<Object> {
 
     private Deferred deferred;
 
-    private AtomicBoolean done = new AtomicBoolean(false);
+    private final AtomicBoolean done = new AtomicBoolean(false);
 
     public DeferredSubscriber(final Deferred deferred) {
       this.deferred = deferred;
@@ -366,7 +366,7 @@ public DeferredSubscriber(final Deferred deferred) {
     @Override
     public void onCompleted() {
       if (done.compareAndSet(false, true)) {
-        deferred.resolve((Object) null);
+        deferred.resolve(null);
       }
       deferred = null;
     }

Original file line number	Diff line number	Diff line change
`@@ -357,7 +357,7 @@ static class DeferredSubscriber extends Subscriber<Object> {`
`357`	`357`
`358`	`358`	`private Deferred deferred;`
`359`	`359`
`360`		`- private AtomicBoolean done = new AtomicBoolean(false);`
	`360`	`+ private final AtomicBoolean done = new AtomicBoolean(false);`
`361`	`361`
`362`	`362`	`public DeferredSubscriber(final Deferred deferred) {`
`363`	`363`	`this.deferred = deferred;`
`@@ -366,7 +366,7 @@ public DeferredSubscriber(final Deferred deferred) {`
`366`	`366`	`@Override`
`367`	`367`	`public void onCompleted() {`
`368`	`368`	`if (done.compareAndSet(false, true)) {`
`369`		`- deferred.resolve((Object) null);`
	`369`	`+ deferred.resolve(null);`
`370`	`370`	`}`
`371`	`371`	`deferred = null;`
`372`	`372`	`}`