Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code analysis stop working (until Reload Window) if a save is done that fails while code analysis is running (and save shouldn't fail) #12721

Open
sean-mcmanus opened this issue Sep 17, 2024 · 2 comments
Assignees
Labels
bug Feature: Code Analysis Related to integration with clang-tidy, cppcheck, cl.exe /analyze, etc. Language Service
Milestone

Comments

@sean-mcmanus
Copy link
Collaborator

  1. Run code analysis on some file.
  2. Make some edit and save to some file that is locked for editing by clang-tidy. You should get a file save failed notification from VS Code.
  3. Cancel code analysis (optional, it should repro the bug even if you wait till it finishes).
  4. Run code analysis again.

Bug: Code analysis never actually runs and is stuck in a running state (internally it's waiting for the save of the file that never arrived). The code expected that a willSaveWaitUntil will always be followed by a save.

@sean-mcmanus sean-mcmanus added bug Language Service Feature: Code Analysis Related to integration with clang-tidy, cppcheck, cl.exe /analyze, etc. labels Sep 17, 2024
@sean-mcmanus sean-mcmanus self-assigned this Sep 17, 2024
@sean-mcmanus
Copy link
Collaborator Author

sean-mcmanus commented Sep 17, 2024

Oh, looks like the "root" (or 2nd) bug is that the save shouldn't fail and there's a problem canceling the code analysis when the willSaveWaitUntil is called.

@sean-mcmanus sean-mcmanus added this to the 1.22 milestone Sep 17, 2024
@sean-mcmanus sean-mcmanus changed the title Code analysis stop working (until Reload Window) if a save is done that fails while code analysis is running Code analysis stop working (until Reload Window) if a save is done that fails while code analysis is running (and save shouldn't fail) Sep 17, 2024
@sean-mcmanus
Copy link
Collaborator Author

Doing a successful save while code analysis is running might be a workaround though (not sure yet)...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Feature: Code Analysis Related to integration with clang-tidy, cppcheck, cl.exe /analyze, etc. Language Service
Projects
Status: Todo
Development

No branches or pull requests

1 participant