feat/items-grammar, closures and goto#13
Conversation
Flecart
force-pushed
the
feat/items-grammar
branch
from
58ba95b to
af713a3
Compare
giospada
left a comment
giospada
left a comment
There was a problem hiding this comment.
riga 88: _first può può andare in loop infinito?
credo proprio di sì, fixo |
|
boh, io cambierei un po' tutto |
|
io creerei una struttura, per i first e i follow e mi atterrei al codice del prof, |
|
anche per ridurrei la lunghezza dei del file che è lunghissimo |
|
per esempio la struttura first si memorizza la tabella dei first |
|
cosa ne dici? |
Ridurre la lunghezza e ordinare un pò le cose sì. |
Flecart
force-pushed
the
feat/items-grammar
branch
from
b08a92b to
b6708cd
Compare
|
si io i first and follow li precalcolerei in una struttura esterna, e poi passerei quella struttura per le altre funzioni a cui serve |
|
l'algoritmo delle dei first and follow lo trovi sulle slide |
* feat(first_follow): first version * test(first_follow): move tests * test(first_follow): remove old implementation of first and follow * test(first_follow): fix tests and first bug
giospada
left a comment
giospada
left a comment
There was a problem hiding this comment.
Io incomincierei a fare un piccolo refactor su questa parte, per esempio non mi piace la first and follow table, le terrei come due cose separate
| Letter::NonTerminal(idx) => self.first_table.as_ref().unwrap()[*idx].clone(), | ||
| Letter::Terminal(ch) => { | ||
| let mut set = BTreeSet::new(); | ||
| set.insert(*ch); |
|
inoltre credo che come prima cosa bisognerebbe mettere aposto l'alfabeto, creare un alfabeto unico che possa essere utilizzato in tutta l'applicazione |
|
Item non sarebbe il caso di metterlo in un altro gruppo di directory tipo LR parser? |
2 participants
No description provided.