search

coli-saar / alto

Alto, the Algebraic Language Toolkit
Other
16 stars 2 forks source link

WideStringAlgebra does not redefine protected evaluate #6

Closed akoehn closed 7 years ago

akoehn commented 9 years ago

Original report by Christoph Teichmann (Bitbucket: cteichmann, GitHub: CTNLP).

The WideStringAlgebra redefines the concatenation symbols to be conc+num, but the evaluate(String,List<List>) method is not redefined, this should be remedied in order to make its behaviour consistent with the decomposition automata that are generated

akoehn commented 7 years ago

Original comment by Christoph Teichmann (Bitbucket: cteichmann, GitHub: CTNLP).

I changed the code so it will interpreted any symbol with children in the evaluate function as a concatenate function, this was necessary since the inherited function evaluated any subtree with a root other than * to a single label.

akoehn commented 9 years ago

Original changes by Christoph Teichmann (Bitbucket: cteichmann, GitHub: CTNLP).

changed content from "The WideStringAlgebra redefines the concatenation symbols to be conc+num, but the evaluate(String,List<List> method is not redefined), this should be remedied in order to make its behaviour consistent with the decomposition automata that are gene" to "The WideStringAlgebra redefines the concatenation symbols to be conc+num, but the evaluate(String,List<List>) method is not redefined, this should be remedied in order to make its behaviour consistent with the decomposition automata that are gene"

akoehn commented 7 years ago

Original changes by Christoph Teichmann (Bitbucket: cteichmann, GitHub: CTNLP).

changed state from "new" to "resolved"