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

Conditional breakpoints and log points cause execution to stop as if it were a normal breakpoint. #2862

Open
makzimus opened this issue Nov 23, 2018 · 5 comments

Comments

@makzimus
Copy link

makzimus commented Nov 23, 2018

Using:
vscode version: 1.29.1
C/c++ extension version: 0.20.1
Launch config using type: "cppvsdbg".

Unsure of what other details to provide.

@pieandcakes
Copy link
Contributor

@Makail can you enable a engineLogging and provide the log please for the conditional breakpoints issue.

Log points are currently not supported and we have a feature request for it, #2591.

@pieandcakes pieandcakes added debugger more info needed The issue report is not actionable in its current state labels Nov 26, 2018
@makzimus
Copy link
Author

makzimus commented Nov 28, 2018

` -> (C) {"command":"setBreakpoints","arguments":{"source":{"name":"filename.cc","path":"e:\filepath\filename.cc","sourceReference":0,"sources":[],"checksums":[{"algorithm":"MD5","checksum":"6bbee315a8e0046953c5fafb314e2e08"}]},"lines":[252],"breakpoints":[{"line":252}],"sourceModified":false},"type":"request","seq":86}
<- (R) {"seq":1563,"type":"response","request_seq":86,"success":true,"command":"setBreakpoints","message":"","body":{"breakpoints":[{"id":5,"verified":true,"source":{"name":"filename.cc","path":"e:\filepath\filename.cc","sourceReference":0,"sources":[],"checksums":[{"algorithm":"MD5","checksum":"6bbee315a8e0046953c5fafb314e2e08"}]},"line":252,"endLine":252}]}}
-> (C) {"command":"setBreakpoints","arguments":{"source":{"name":"filename.cc","path":"e:\filepath\filename.cc","sourceReference":0,"sources":[],"checksums":[{"algorithm":"MD5","checksum":"6bbee315a8e0046953c5fafb314e2e08"}]},"lines":[252],"breakpoints":[{"line":252,"condition":"diff > mScrollThreshold"}],"sourceModified":false},"type":"request","seq":87}
<- (R) {"seq":1566,"type":"response","request_seq":87,"success":true,"command":"setBreakpoints","message":"","body":{"breakpoints":[{"id":5,"verified":true,"source":{"name":"filename.cc","path":"e:\filepath\filename.cc","sourceReference":0,"sources":[],"checksums":[{"algorithm":"MD5","checksum":"6bbee315a8e0046953c5fafb314e2e08"}]},"line":252,"endLine":252}]}}
<- (E) {"seq":1568,"type":"event","event":"stopped","body":{"reason":"breakpoint","threadId":89100,"allThreadsStopped":true,"source":{"name":"filename.cc","path":"e:\filepath\filename.cc","sources":[],"checksums":[]},"line":252,"column":0}}
-> (C) {"command":"threads","type":"request","seq":88}

...

-> (C) {"command":"evaluate","arguments":{"expression":"diff","frameId":1000,"context":"hover"},"type":"request","seq":94}
<- (R) {"seq":1591,"type":"response","request_seq":94,"success":true,"command":"evaluate","message":"","body":{"result":"0.000000000","type":"float","presentationHint":{"kind":"data","attributes":[]},"variablesReference":0}}
-> (C) {"command":"evaluate","arguments":{"expression":"mScrollThreshold","frameId":1000,"context":"hover"},"type":"request","seq":95}
<- (R) {"seq":1594,"type":"response","request_seq":95,"success":true,"command":"evaluate","message":"","body":{"result":"40.0000000","type":"float","presentationHint":{"kind":"data","attributes":[],"visibility":"private"},"variablesReference":0}} `

@pieandcakes
Copy link
Contributor

Thanks. I'll update the issue as investigate

@pieandcakes pieandcakes added investigate: repro This issue's repro steps needs to be investigated/confirmed and removed more info needed The issue report is not actionable in its current state labels Nov 28, 2018
@aenima1891
Copy link

aenima1891 commented Aug 28, 2019

I've the same problem.
There are some news about it?

@pieandcakes pieandcakes added Feature Request and removed investigate: repro This issue's repro steps needs to be investigated/confirmed labels Aug 28, 2019
@pieandcakes
Copy link
Contributor

For conditional breakpoints, we have this issue for tracking it which you can upvote. Logpoints are currently not supported in the extension so they are treated like a breakpoint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants