MultiSourceLine.getSourceAndLineFromOverallLine() generates a lot of objects

[andimarek]

[bbakerman]

It has to track three things - a column, and line number and the source name that the text came from

It allocates a SourceAndLine sourceAndLine = new SourceAndLine(); and then a new SourceLocation(line, column, sourceAndLine.getSourceName()); for every AST node - so 2 per AST node

How would you do it otherwise? Move the source and col and int INTO the AST nodes themselves

[bbakerman]

SourceAndLine should have a very short life span - SourceLocation lives as long as the AST node

[github-actions[bot]]

Hello, this issue has been inactive for 60 days, so we're marking it as stale. If you would like to continue this discussion, please comment within the next 30 days or we'll close the issue.