Skip to content

Commit 97647c6

Browse files
committed
Fixed style issues.
1 parent ba68c84 commit 97647c6

File tree

6 files changed

+12
-20
lines changed

6 files changed

+12
-20
lines changed

wasm/src/org.graalvm.wasm/src/org/graalvm/wasm/debugging/data/DebugType.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,6 +98,7 @@ public boolean isModifiableValue() {
9898
*
9999
* @see #isModifiableValue()
100100
*/
101+
@SuppressWarnings("unused")
101102
public void setValue(DebugContext context, DebugLocation location, Object value, InteropLibrary lib) {
102103
}
103104

wasm/src/org.graalvm.wasm/src/org/graalvm/wasm/debugging/parser/DebugParser.java

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -833,16 +833,6 @@ private short read2() throws WasmDebugException {
833833
}
834834
}
835835

836-
/**
837-
* Reads two bytes as an unsigned int value from the internal byte array and advances the offset
838-
* pointer.
839-
*
840-
* @throws WasmDebugException if the data is beyond the current endOffset.
841-
*/
842-
private int readUnsigned2() throws WasmDebugException {
843-
return read2() & 0xffff;
844-
}
845-
846836
/**
847837
* Reads four bytes as an int value from the internal byte array without advancing the offset
848838
* pointer.

wasm/src/org.graalvm.wasm/src/org/graalvm/wasm/debugging/parser/DebugTranslator.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ public EconomicMap<Integer, DebugFunction> readCompilationUnits(byte[] customDat
7676
if (context != null) {
7777
final DebugData compilationUnit = parser.readCompilationUnitChildren(unit, debugInfoOffset);
7878
if (compilationUnit != null) {
79-
if (parseFunctions(context, compilationUnit)) {
79+
if (DebugTranslator.parseFunctions(context, compilationUnit)) {
8080
debugFunctions.putAll(context.functions());
8181
}
8282
}
@@ -133,7 +133,7 @@ private DebugParserContext parseCompilationUnit(DebugParseUnit parseUnit, byte[]
133133
return new DebugParserContext(customData, debugInfoOffset, entries, fileLineMaps, filePaths, languageName, objectFactory);
134134
}
135135

136-
private boolean parseFunctions(DebugParserContext context, DebugData data) {
136+
private static boolean parseFunctions(DebugParserContext context, DebugData data) {
137137
final int[] pcs = DebugDataUtil.readPcsOrNull(data, context);
138138
if (pcs == null) {
139139
return false;

wasm/src/org.graalvm.wasm/src/org/graalvm/wasm/debugging/representation/DebugObjectDisplayValue.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -132,8 +132,8 @@ Object readMember(String member) throws UnknownIdentifierException {
132132
@TruffleBoundary
133133
Object getMembers(@SuppressWarnings("unused") boolean includeInternal) {
134134
final List<String> names = new ArrayList<>(members.size());
135-
for (String name : members.getKeys()) {
136-
names.add(name);
135+
for (String member : members.getKeys()) {
136+
names.add(member);
137137
}
138138
return new WasmVariableNamesObject(names);
139139
}

wasm/src/org.graalvm.wasm/src/org/graalvm/wasm/debugging/representation/DebugScopeDisplayValue.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,7 @@
6666
import com.oracle.truffle.api.source.SourceSection;
6767

6868
@ExportLibrary(InteropLibrary.class)
69+
@SuppressWarnings("static-method")
6970
public final class DebugScopeDisplayValue extends DebugDisplayValue implements TruffleObject {
7071
private final String name;
7172
private final DebugContext context;
@@ -169,8 +170,8 @@ Object readMember(String member) throws UnknownIdentifierException {
169170

170171
private List<String> memberNames() {
171172
final List<String> names = new ArrayList<>(0);
172-
for (String name : members.getKeys()) {
173-
names.add(name);
173+
for (String member : members.getKeys()) {
174+
names.add(member);
174175
}
175176
return names;
176177
}

wasm/src/org.graalvm.wasm/src/org/graalvm/wasm/nodes/WasmInstrumentationSupportNode.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -87,11 +87,11 @@ public void notifyLine(VirtualFrame frame, int currentLineIndex, int nextLineInd
8787
return;
8888
}
8989
this.sourceLocation = currentSourceLocation;
90-
exitAt(frame, currentLineWrapper);
91-
enterAt(frame, nextLineWrapper);
90+
WasmInstrumentationSupportNode.exitAt(frame, currentLineWrapper);
91+
WasmInstrumentationSupportNode.enterAt(frame, nextLineWrapper);
9292
}
9393

94-
private void enterAt(VirtualFrame frame, InstrumentableNode.WrapperNode wrapperNode) {
94+
private static void enterAt(VirtualFrame frame, InstrumentableNode.WrapperNode wrapperNode) {
9595
if (wrapperNode == null) {
9696
return;
9797
}
@@ -110,7 +110,7 @@ private void enterAt(VirtualFrame frame, InstrumentableNode.WrapperNode wrapperN
110110
}
111111
}
112112

113-
private void exitAt(VirtualFrame frame, InstrumentableNode.WrapperNode wrapperNode) {
113+
private static void exitAt(VirtualFrame frame, InstrumentableNode.WrapperNode wrapperNode) {
114114
if (wrapperNode == null) {
115115
return;
116116
}

0 commit comments

Comments
 (0)