rfe: Substring doesn't need the string as a parameter

RESOLVED FIXED in mozilla0.9

Status

()

Core
String
RESOLVED FIXED
18 years ago
18 years ago

People

(Reporter: gordon, Assigned: Scott Collins)

Tracking

Trunk
mozilla0.9
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

18 years ago
Since Substring has two iterators, the string parameter appears to be redundant.  
Is it possible to eliminate it?
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9
(Assignee)

Comment 1

18 years ago
Created attachment 26066 [details] [diff] [review]
drops extra parameter, un-templatizes |Substring()|, and fixes callers
(Assignee)

Comment 2

18 years ago
this patch builds and runs on Mac; LXR didn't reveal any other callers to me
Keywords: approval, patch, review
(Assignee)

Updated

18 years ago
OS: Mac System 8.5 → All
(Assignee)

Comment 3

18 years ago
This patch also fixes bug #69289.
(Assignee)

Comment 5

18 years ago
patch checked in
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
(Assignee)

Updated

18 years ago
Keywords: approval, patch, review
(Assignee)

Updated

18 years ago
Blocks: 70090
You need to log in before you can comment on or make changes to this bug.