Skip to content

VM validation step redundancy (MODERATE) #3

@mrhapile

Description

@mrhapile

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

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingenhancementNew feature or request

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions