Skip to content

Commit 47e4284

Browse files
committed
Sonarqube: Reduce visibility of Object[] array and provide copying accessor
1 parent e583b7b commit 47e4284

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/traceback/PTraceback.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,11 @@ public final class PTraceback extends PythonBuiltinObject {
5454
public static final String TB_LASTI = "tb_lasti";
5555
public static final String TB_LINENO = "tb_lineno";
5656

57-
@CompilationFinal(dimensions = 1) public static final Object[] TB_DIR_FIELDS = new Object[]{TB_FRAME, TB_NEXT, TB_LASTI, TB_LINENO};
57+
@CompilationFinal(dimensions = 1) private static final Object[] TB_DIR_FIELDS = new Object[]{TB_FRAME, TB_NEXT, TB_LASTI, TB_LINENO};
58+
59+
static Object[] getTbFieldNames() {
60+
return TB_DIR_FIELDS.clone();
61+
}
5862

5963
private final PBaseException exception;
6064
private final int index;

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/traceback/TracebackBuiltins.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@
2525
*/
2626
package com.oracle.graal.python.builtins.objects.traceback;
2727

28-
import static com.oracle.graal.python.builtins.objects.traceback.PTraceback.TB_DIR_FIELDS;
2928
import static com.oracle.graal.python.builtins.objects.traceback.PTraceback.TB_FRAME;
3029
import static com.oracle.graal.python.builtins.objects.traceback.PTraceback.TB_LASTI;
3130
import static com.oracle.graal.python.builtins.objects.traceback.PTraceback.TB_LINENO;
@@ -57,7 +56,7 @@ protected List<? extends NodeFactory<? extends PythonBuiltinBaseNode>> getNodeFa
5756
public abstract static class DirNode extends PythonBuiltinNode {
5857
@Specialization
5958
public Object dir(@SuppressWarnings("unused") PTraceback self) {
60-
return factory().createList(TB_DIR_FIELDS.clone());
59+
return factory().createList(PTraceback.getTbFieldNames());
6160
}
6261
}
6362

0 commit comments

Comments
 (0)