[refactor] remove redundant check

Author: kares

src/main/java/org/jruby/ext/openssl/BN.java

@@ -909,7 +909,7 @@ public static BigInteger getBigInteger(final IRubyObject arg) {
 
     @Override
     public Object toJava(Class target) {
-        if ( target.isAssignableFrom(BigInteger.class) || target == Number.class ) return value;
+        if ( target.isAssignableFrom(BigInteger.class) ) return value;
         if ( target == Long.class || target == Long.TYPE ) return value.longValue();
         if ( target == Integer.class || target == Integer.TYPE ) return value.intValue();
         if ( target == Double.class || target == Double.TYPE ) return value.doubleValue();

src/test/ruby/test_bn.rb

@@ -78,7 +78,9 @@ def test_comparison
 
   def test_to_java
     assert_equal java.lang.Integer.new(42), OpenSSL::BN.new('42').to_java(:int)
-    assert_equal java.math.BigInteger.valueOf(24), OpenSSL::BN.new('24').to_java
+    assert_equal java.math.BigInteger.valueOf(24), val = OpenSSL::BN.new('24').to_java
+    assert_equal java.math.BigInteger, val.class
+    assert_equal java.math.BigInteger.valueOf(24), val = OpenSSL::BN.new('24').to_java(java.lang.Number)
   end if defined? JRUBY_VERSION
 
   def test_new_str