-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Dynamic debug config info #8861
Conversation
@@ -31,15 +31,15 @@ function isDebugLaunchStr(str: string): boolean { | |||
* Ensures that the selected configuration's preLaunchTask (if existent) is populated in the user's task.json. | |||
* Automatically starts debugging for "Build and Debug" configurations. | |||
*/ | |||
export class QuickPickConfigurationProvider implements vscode.DebugConfigurationProvider { | |||
private underlyingProvider: CppConfigurationProvider; | |||
export class CppDebugConfigurationProvider implements vscode.DebugConfigurationProvider { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's not rename all this stuff. Can you change it back?
Naming a class "Underlying" isn't a very helpful choice. "Underlying" only has meaning within the context that it's used. A standalone class is not "underlying" anything.
We shouldn't use the abbreviation dbg
in variable names. In general abbreviations should be avoided, but in this case it also causes confusion with the cppdbg
debug type which is an actual thing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the name underlying is actually currently existing, I removed it in the next PR, in refactoring the code.
…/vscode-cpptools into elrashed/dynamicLaunch
Modification to "Run and Debug" button in debug panel to make it consistent with our play button:
3.Fixes are added for single-mode file debugging, however it is only working for cl.exe builds and msvc debugger for now
Feature Request: #8773