nitrix/klang
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

use GetStructNames instead of collecting the names in GetStructs

Browse Source
This commit is contained in:
nitrix
2020-02-04 21:44:04 +01:00
parent a79a2332a1
commit 19daaa63af
2 changed files with 8 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/main/java/de/hsrm/compiler/Klang/GetStructs.java
View File

@@ -2,7 +2,6 @@ package de.hsrm.compiler.Klang;
import java.util.Map;
import java.util.Set;
import java.util.HashSet;
import de.hsrm.compiler.Klang.helper.Helper;
import de.hsrm.compiler.Klang.nodes.Node;
@@ -16,9 +15,9 @@ public class GetStructs extends KlangBaseVisitor<Node> {
private Set<String> structNames;
private Map<String, StructDefinition> structs;
public GetStructs(Map<String, StructDefinition> structs) {
public GetStructs(Set<String> structNames, Map<String, StructDefinition> structs) {
this.structs = structs;
this.structNames = new HashSet<>();
this.structNames = structNames;
}
@Override
@@ -36,13 +35,6 @@ public class GetStructs extends KlangBaseVisitor<Node> {
int line = ctx.start.getLine();
int col = ctx.start.getCharPositionInLine();
StructField[] fields = new StructField[ctx.structField().size()];
if (this.structNames.contains(name)) {
String error = "Struct " + name + " defined multiple times.";
throw new Error(Helper.getErrorPrefix(line, col) + error);
}
this.structNames.add(name);
for (int i = 0; i < ctx.structField().size(); i++) {
StructField field = (StructField) this.visit(ctx.structField(i));

7
src/main/java/de/hsrm/compiler/Klang/Klang.java
View File

@@ -8,6 +8,7 @@ import java.io.*;
import java.util.Arrays;
import java.util.List;
import java.util.HashMap;
import java.util.HashSet;
import de.hsrm.compiler.Klang.nodes.Node;
import de.hsrm.compiler.Klang.nodes.StructDefinition;
@@ -91,9 +92,13 @@ public class Klang {
var functionDefinitions = new HashMap<String, FunctionInformation>();
new GetFunctions(functionDefinitions).visit(tree);
// Extract names of all structs
var structNames = new HashSet<String>();
new GetStructNames(structNames).visit(tree);
// Extract information about all structs
var structs = new HashMap<String, StructDefinition>();
new GetStructs(structs).visit(tree);
new GetStructs(structNames, structs).visit(tree);
// Create the DAST
ContextAnalysis ctxAnal = new ContextAnalysis(functionDefinitions, structs);