Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat(treesitter): use better error messages in query#17984

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
vigoux wants to merge1 commit intoneovim:masterfromvigoux:better_query_errors

Conversation

@vigoux
Copy link
Contributor

This PR is an attempt to return better error messages when parsing
queries.

Somehow I can't handle the error object from the lua side.
It is even weirder: when returning any error from the C-side, it is
not catched bypcall after, norxpcall...

TODO:

  • Actually handle the message on the lua side

Superseedes#14053

WhyNotHugo and shazow reacted with thumbs up emoji
self.captures=self.info.captures
query_cache[lang][query]=self
returnself
print("OKAY")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

leftover debug message?

@vigoux
Copy link
ContributorAuthor

@bfredl I might need you insights here, because I can't find any way to catch the error from the C side on the lua side.

I wonder if I should just mimic apcall from the C side, returning true / false and the object depending on the result.

@clason
Copy link
Member

I think this is now itself superseded :)

@clasonclason closed thisSep 17, 2023
@github-actionsgithub-actionsbot removed the request for review frombfredlSeptember 17, 2023 09:58
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@clasonclasonclason left review comments

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vigoux@clason@zeertzjq

[8]ページ先頭

©2009-2025 Movatter.jp