Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

Allow for recursive queries #1377

Open
Xanewok opened this issue Feb 25, 2019 · 1 comment
Open

Allow for recursive queries #1377

Xanewok opened this issue Feb 25, 2019 · 1 comment
Labels
enhancement Indicates new feature requests package: rls-analysis

Comments

@Xanewok
Copy link
Member

Xanewok commented Feb 25, 2019

Originally created by @Xanewok (rust-dev-tools/rls-analysis#152):

Right now this leads to a silent deadlock because every public query is wrapped in a self.with_analysis(|a| ...) which locks the primary lock for the execution time of the query.

One place where recursive queries are handy is recursive symbol tree processing, e.g. when reconstructing hierarchical symbol outline.

I believe it'd be good to be a bit more explicit about the interior lock and expose it so the user can perform multiple queries in bulk.

@jplatte
Copy link
Contributor

jplatte commented Sep 29, 2019

Is this something that requires extensive knowledge about RLS or could a new contributor potentially implement it?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Indicates new feature requests package: rls-analysis
Projects
None yet
Development

No branches or pull requests

2 participants