Vector35 / Vector35/binaryninja-api
Dead Store Elimination Context Menu Rant
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 1.3k
- Forks
- 298
- Avg merge
- 5d 5h
- Merged PRs (30d)
- 19
Description
Is this a general concept that needs to be documented or a specific API?
What is the purpose of two other options except "Allow" in Dead Store Elimination ?
What is the purpose of Default if it does the same thing as Prevent ?
Can't this be simplified to one action with Remove unused variable or something similar ? Since if we remove the variable with DSE there is no way of getting it back (can't change removed variable DSE to Prevent or Default because it's not even visible, or at least manual does not explain it)
I've read the documentation about DSE which is short, which makes sense because it's a small action, but why does it need 3 options and a dedicated submenu in context menu ? I don't know.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No file or test is named. Start by reviewing the Dead Store Elimination documentation and its context-menu options, especially Allow, Default, and Prevent; done should clearly explain their purposes and whether the requested menu simplification is intended.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- reverse-engineering
- Issue type
- Documentation
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100