Closed Bug 940558 Opened 10 years ago Closed 10 years ago

Fix an exact rooting false positive in replaceLastProperty

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla28

People

(Reporter: terrence, Assigned: terrence)

References

Details

(Whiteboard: [qa-])

Attachments

(1 file)

hazard_replaceLastProperty-v0.diff
1.56 KB, patch
terrence
: review+
Details | Diff | Splinter Review 
We can trivially reorder the statements here to make an potential hazard go away.

Over-the-shoulder r+ from sfink.
Attachment #8334694 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/bc13933a8f76
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.