Status

()

RESOLVED INVALID
18 years ago
18 years ago

People

(Reporter: jgmyers, Assigned: scc)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

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

Updated

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

Comment 1

18 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: 18 years ago
Resolution: --- → INVALID
(Assignee)

Comment 3

18 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.