Change signature of Emit1 et al to avoid silly idioms

NEW
Unassigned

Status

()

Core
JavaScript Engine
4 years ago
3 years ago

People

(Reporter: jorendorff, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

4 years ago
Most Emit1 call sites only use the result to test for success/failure.

It's error-prone. Out param for the call sites that need it.

While we're on the subject, these should probably be methods of BytecodeEmitter.
(Assignee)

Updated

3 years ago
Assignee: general → nobody
You need to log in before you can comment on or make changes to this bug.