If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Status

()

Core
String
RESOLVED INVALID
17 years ago
17 years ago

People

(Reporter: John G. Myers, Assigned: Scott Collins)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

17 years ago
CopyUTF8toUCS2() is absent from nsReadableUtils.h
CopyUCS2toUTF8() should be added as well.
(Assignee)

Updated

17 years ago
Blocks: 73009
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.0
(Assignee)

Comment 1

17 years ago
re-targeting milestones, starting from a clean slate
Target Milestone: mozilla1.0 → ---
Turns out that this is not needed now that we have Substring that has a
constructor that takes two iterators, as scc explained to me. Making INVALID.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → INVALID
(Assignee)

Comment 3

17 years ago
In case anyone cares, the signature for one of thise copy functions would have been

  CopyXXXtoYYY( sourceStartIter, sourceEndIter, destString );

but this is no better than saying

  destString = Substring(sourceStartIter, sourceEndIter);

in fact ... it's worse in the current implementations of the |CopyXXXtoYYY|s
that do exist, since they don't check for dependencies as assignment does.  I've
filed bug #81433 to get rid of the existing iterator-based |CopyXXXtoYYY| string
functions (|CopyUnicodeTo| and |AppendUnicodeTo|).
You need to log in before you can comment on or make changes to this bug.