[refactor] some minor cleanup left-overs

kares · kares · commit 2a38f3a4a95e · 2018-06-22T15:34:44.000+02:00
diff --git a/src/main/java/org/jruby/ext/openssl/SSLSocket.java b/src/main/java/org/jruby/ext/openssl/SSLSocket.java
@@ -428,12 +428,11 @@ private IRubyObject fallback_verify_mode(final ThreadContext context) {
 
     @JRubyMethod
     public IRubyObject verify_result(final ThreadContext context) {
-        final Ruby runtime = context.runtime;
         if (engine == null) {
-            runtime.getWarnings().warn("SSL session is not started yet.");
-            return runtime.getNil();
+            context.runtime.getWarnings().warn("SSL session is not started yet.");
+            return context.nil;
         }
-        return runtime.newFixnum(verifyResult);
+        return context.runtime.newFixnum(verifyResult);
     }
 
     // This select impl is a copy of RubyThread.select, then blockingLock is

Original file line number	Diff line number	Diff line change
`@@ -428,12 +428,11 @@ private IRubyObject fallback_verify_mode(final ThreadContext context) {`
`428`	`428`
`429`	`429`	`@JRubyMethod`
`430`	`430`	`public IRubyObject verify_result(final ThreadContext context) {`
`431`		`- final Ruby runtime = context.runtime;`
`432`	`431`	`if (engine == null) {`
`433`		`- runtime.getWarnings().warn("SSL session is not started yet.");`
`434`		`- return runtime.getNil();`
	`432`	`+ context.runtime.getWarnings().warn("SSL session is not started yet.");`
	`433`	`+ return context.nil;`
`435`	`434`	`}`
`436`		`- return runtime.newFixnum(verifyResult);`
	`435`	`+ return context.runtime.newFixnum(verifyResult);`
`437`	`436`	`}`
`438`	`437`
`439`	`438`	`// This select impl is a copy of RubyThread.select, then blockingLock is`