System.gc() is async, do not attempt to invoke it and retry sysopen a…

…s it will always fail
jruby · Nov 2, 2016 · 0bc03a9 · 0bc03a9
1 parent e3cc519
commit 0bc03a9
Showing 1 changed file with 4 additions and 10 deletions.
diff --git a/core/src/main/java/org/jruby/RubyIO.java b/core/src/main/java/org/jruby/RubyIO.java
@@ -1213,16 +1213,10 @@ protected static ChannelFD sysopen(Ruby runtime, String fname, int oflags, int p
 
         fd = sysopenInternal(runtime, data);
         if (fd == null) {
-            if (data.errno == Errno.EMFILE || data.errno == Errno.ENFILE) {
-                System.gc();
-                data.errno = null;
-                fd = sysopenInternal(runtime, data);
-            }
-            if (fd == null) {
-                if (data.errno != null) {
-                    throw runtime.newErrnoFromErrno(data.errno, fname);
-                }
-                throw runtime.newSystemCallError(fname);
+            if (data.errno != null) {
+                throw runtime.newErrnoFromErrno(data.errno, fname.toString());
+            } else {
+                throw runtime.newSystemCallError(fname.toString());
             }
         }
         return fd;