Vector should have const and non-const begin()

RESOLVED FIXED in mozilla12

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jorendorff, Assigned: jorendorff)

Tracking

Other Branch
mozilla12
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Vector has both const and non-const end() methods but only a const begin() method, and it returns a pointer-to-non-const T, which seems wrong to me.

Comment 1

6 years ago
Indeed, Vector used to have this right; it looks like it regressed in http://hg.mozilla.org/mozilla-central/rev/b60ac564e39d#l1.253.
(Assignee)

Comment 2

6 years ago
Created attachment 587745 [details] [diff] [review]
v1
Assignee: general → jorendorff
Attachment #587745 - Flags: review?(luke)

Updated

6 years ago
Attachment #587745 - Flags: review?(luke) → review+
https://hg.mozilla.org/mozilla-central/rev/11ba93db719a
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
You need to log in before you can comment on or make changes to this bug.