Closed Bug 505901 Opened 11 years ago Closed 11 years ago

PutProperty has useless null check in !targetprop else case

Categories

(Core :: JavaScript Engine, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: timeless, Assigned: timeless)

References

(Blocks 1 open bug, )

Details

(Keywords: coverity)

Attachments

(1 file)

4162             if (!targetprop || IS_STAR(GetLocalName(targetprop))) {
4167             } else {
4168                 nameobj = targetprop;
4169                 if (!nameobj)
4170                     goto bad;
Attached patch remove themSplinter Review
Assignee: general → timeless
Status: NEW → ASSIGNED
Attachment #390311 - Flags: review?(gal)
Attachment #390311 - Flags: review?(gal) → review+
http://hg.mozilla.org/mozilla-central/rev/68707c364290
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.