Skip to content

Cleanup Transformer #234

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
MaillPierre merged 2 commits into from
Nov 20, 2025
Merged

Cleanup Transformer #234

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
5b3ba0b
Transformer: commented out 12 methods + useless imports
Nov 18, 2025
fe62b70
Deleting commented dead code
MaillPierre Nov 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
68 changes: 0 additions & 68 deletions src/main/java/fr/inria/corese/core/compiler/parser/Transformer.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
import fr.inria.corese.core.kgram.core.Query;
import fr.inria.corese.core.kgram.core.*;
import fr.inria.corese.core.sparql.api.IDatatype;
import fr.inria.corese.core.sparql.compiler.java.JavaCompiler;
import fr.inria.corese.core.sparql.exceptions.EngineException;
import fr.inria.corese.core.sparql.exceptions.SafetyException;
import fr.inria.corese.core.sparql.triple.cst.RDFS;
Expand All @@ -21,7 +20,6 @@
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import java.io.IOException;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
Expand Down Expand Up @@ -113,16 +111,6 @@ public class Transformer implements ExpType {
functionCompiler = new FunctionCompiler(this);
}

Transformer(CompilerFactory f) {
this();
fac = f;
compiler = fac.newInstance();
}

public static Transformer create(CompilerFactory f) {
return new Transformer(f);
}

public static Transformer create() {
return new Transformer();
}
Expand Down Expand Up @@ -388,20 +376,6 @@ void metadata(ASTQuery ast, Query q) {
}
}

@Deprecated
void toJava(ASTQuery ast) throws EngineException {
if (ast.hasMetadata(Metadata.Type.COMPILE)) {
String name = ast.getMetadata().getValue(Metadata.Type.COMPILE);
JavaCompiler jc = new JavaCompiler(name);
try {
jc.compile(ast);
jc.write();
} catch (IOException ex) {
logger.error(ex.getMessage());
}
}
}

void visit(ASTQuery ast) {
visitor(ast);
if (getVisitorList() != null) {
Expand Down Expand Up @@ -581,11 +555,6 @@ void error(Query q, ASTQuery ast) throws EngineException {
getFunctionCompiler().undefinedFunction(q, ast, ast.getLevel());
}


public void imports(Query q, String path) throws EngineException {
getFunctionCompiler().imports(q, q.getAST(), path);
}

public boolean getLinkedFunction(String label) throws EngineException {
return getFunctionCompiler().getLinkedFunction(label);
}
Expand Down Expand Up @@ -851,14 +820,6 @@ Exp compileService(Service service) throws EngineException {
return exp;
}

Query create(Exp exp) {
Query q = Query.create(exp);
if (sort != null) {
q.set(sort);
}
return q;
}

Exp compileValues(Values val, boolean opt, int level) {
Exp exp = compileValues(val);
if (exp == null) {
Expand Down Expand Up @@ -968,10 +929,6 @@ Mapping create(Node[] lNode, List<Node> lVal) {
return Mapping.safeCreate(lNode, nodes);
}

Exp construct(ASTQuery ast) throws EngineException {
return compile(ast, ast.getInsert());
}

Exp delete(ASTQuery ast) throws EngineException {
return compile(ast, ast.getDelete());
}
Expand All @@ -984,10 +941,6 @@ public void setAST(ASTQuery ast) {
this.ast = ast;
}

public Compiler getCompiler() {
return compiler;
}

void complete(Query qCurrent, ASTQuery ast) throws EngineException {
qCurrent.collect();
select(qCurrent, ast);
Expand Down Expand Up @@ -1199,10 +1152,6 @@ Node getNode(Query qCurrent, Variable variable) {
return node;
}

ASTQuery getAST(Query q) {
return q.getAST();
}

Node getProperAndSubSelectNode(Query q, String name) {
Node node;
if (Query.test) {
Expand Down Expand Up @@ -1834,19 +1783,6 @@ public FunctionCompiler getFunctionCompiler() {
return functionCompiler;
}

public void setFunctionCompiler(FunctionCompiler functionCompiler) {
this.functionCompiler = functionCompiler;
}


public int getNumber() {
return number;
}

public void setNumber(int number) {
this.number = number;
}

int incrNumber() {
return number++;
}
Expand Down Expand Up @@ -1899,10 +1835,6 @@ public void setVisitorList(List<QueryVisitor> visit) {
this.visit = visit;
}

public boolean isSPARQLCompliant() {
return isSPARQLCompliant;
}

public void setSPARQLCompliant(boolean b) {
isSPARQLCompliant = b;
}
Expand Down