WebAssembly / WebAssembly/wabt
Remove RunResult
Open
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 8.1k
- Forks
- 827
- Avg merge
- 4d 6h
- Merged PRs (30d)
- 18
Description
wouldn't it be better to just remove RunResult altogether? no RunResult, no bugs!
(more seriously: there are 2 different ways to convey traps currently. would it be feasible to unify them?)
consider this a "TODO" item. (if you could apply a label to this issue, that would be neat.)
Contributor guide
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
Start by locating RunResult and the two existing ways traps are conveyed. Determine whether they can be unified or whether RunResult can be removed; done means the trap behavior is consistent without introducing new bugs.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- compilers
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100