nodejs / llnode

An lldb plugin for Node.js and V8, which enables inspection of JavaScript states for insights into Node.js processes and their core dumps.
Other
1.15k stars 99 forks source link

`unused-but-set-variable` warning in GCC #299

Closed mmarchini closed 4 years ago

mmarchini commented 4 years ago

unused-but-set-variable introduced in b53010b#diff-86a9160147028a03800f8ba4fa555285R1502-R1525 -- not sure how should be handled.

That looks wrong... I'm pretty sure I stumbled on this piece of code a few times and didn't change for a reason. I'll open an issue to investigate it further.

Originally posted by @mmarchini in https://github.com/nodejs/llnode/pull/297#issuecomment-534589385