-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
bugSomething isn't workingSomething isn't workingenhancementNew feature or requestNew feature or request
Description
Observation
In cmdInstantiate() you do:
WasmEdge_VMLoadWasmFromFile()
WasmEdge_VMValidate()
WasmEdge_VMInstantiate()
This is correct, but:
VMInstantiate() already implies validation
Calling VMValidate() explicitly is optional
Why this matters
Not a bug β but reviewers may ask why both are used.
β Recommendation (not mandatory)
Either:
keep it and document why, or
remove VMValidate() for minimalism
π Severity: Moderate
π Phase impact: Phase 4 polish
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingenhancementNew feature or requestNew feature or request