Skip to content

Commit a2b7680

Browse files
authored
Merge pull request jruby#8810 from NGRsoftlab/Fix-8809
Possible fix for 8809
2 parents caac478 + 6f87cc4 commit a2b7680

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

core/src/main/java/org/jruby/runtime/ThreadContext.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -908,7 +908,7 @@ public IRubyObject createCallerBacktrace(int level, int length, Stream<StackWalk
908908
* @param length the length of the trace
909909
* @return an Array with the backtrace locations
910910
*/
911-
public IRubyObject createCallerLocations(int level, Integer length, Stream<StackWalker.StackFrame> stackStream) {
911+
public IRubyObject createCallerLocations(int level, int length, Stream<StackWalker.StackFrame> stackStream) {
912912
runtime.incrementCallerCount();
913913

914914
RubyStackTraceElement[] fullTrace = getPartialTrace(level, length, stackStream);
@@ -924,7 +924,7 @@ public IRubyObject createCallerLocations(int level, Integer length, Stream<Stack
924924
}
925925

926926
/**
927-
* Like {@link #createCallerLocations(int, Integer, Stream)} but accepts a lambda to yield each location and yields
927+
* Like {@link #createCallerLocations(int, int, Stream)} but accepts a lambda to yield each location and yields
928928
* all stack elements until the loop ends or is broken early.
929929
*
930930
* @param stackStream the stream of StackFrame objects from JVM
@@ -934,8 +934,8 @@ public void eachCallerLocation(Stream<StackWalker.StackFrame> stackStream, Consu
934934
eachPartialTrace(stackStream, (elt) -> consumer.accept(RubyThread.Location.newLocation(runtime, elt)));
935935
}
936936

937-
private RubyStackTraceElement[] getPartialTrace(int level, Integer length, Stream<StackWalker.StackFrame> stackStream) {
938-
if (length != null && length == 0) return RubyStackTraceElement.EMPTY_ARRAY;
937+
private RubyStackTraceElement[] getPartialTrace(int level, int length, Stream<StackWalker.StackFrame> stackStream) {
938+
if (length == 0) return RubyStackTraceElement.EMPTY_ARRAY;
939939
return TraceType.Gather.CALLER.getBacktraceData(this, stackStream).getPartialBacktrace(runtime, level + length);
940940
}
941941

@@ -947,7 +947,7 @@ private RubyStackTraceElement[] getWarnTrace(int level, Stream<StackWalker.Stack
947947
return TraceType.Gather.WARN.getBacktraceData(this, stackStream).getPartialBacktrace(runtime, level + 1);
948948
}
949949

950-
private static int safeLength(int level, Integer length, RubyStackTraceElement[] trace) {
950+
private static int safeLength(int level, int length, RubyStackTraceElement[] trace) {
951951
final int baseLength = trace.length - level;
952952
return Math.min(length, baseLength);
953953
}

0 commit comments

Comments
 (0)