search

chappertron / lammps-analyser

A linter for the LAMMPS scripting language
GNU General Public License v2.0
0 stars 0 forks source link

Consolidate "CommandNode" and "CommandType" Types #15

Closed chappertron closed 1 day ago

chappertron commented 1 month ago

All the variants of ast::CommandType have spans associated with them, so the ast::CommandNode is not needed. ast::CommandNode is just a wrapper with a span added:


struct CommandNode {
    type: CommandType,
    span: Span
}

Doing this will make the ASTs significantly easier to use, removeing the need to extract the actual type from the command node,

chappertron commented 1 day ago

This has been done. Not sure when.