Strumenta / kolasu

Kotlin Language Support – AST Library
Apache License 2.0
109 stars 11 forks source link

LionWebModelConverter nodes mapping #318

Closed alessiostalla closed 4 months ago

alessiostalla commented 4 months ago

Sometimes nodesMapping.byA or nodesMapping.byB returns null even when the maps aren't shared across threads. It could be a problem in the application (how the AST is defined).

ftomassetti commented 4 months ago

I would need to investigate this, as I am not sure what could be causing that. I think these maps are based on identity, so it should not be a problem caused by hashcode or equals being depending on some values being changes between insertion and retrieval

alessiostalla commented 4 months ago

This was caused by the client being shared among threads, see https://github.com/Strumenta/starlasu-lionweb-repository-client/issues/22