erik suggested that aLength should mean "the number of bytes in this string that you want to convert" as opposed to "the number of characters in this string that you want to convert". This would make it consistent with the way the Unicode encoders in I18n work. Thought I'd put it into a bug for posterity's sake.
Created attachment 9552 [details] [diff] [review] proposed fix, which treats aLength as bytes count, not character count
scc: this is eventually going to be code that will work via nsCharTraits. Can you make the call wrt. whether or not you want aLength to mean characters or bytes?
scc, i'm giving this to you to decide what's right.
Assignee: waterson → scc
Target Milestone: mozilla0.9 → mozilla0.9.2
re-targeting milestones, starting from a clean slate
Target Milestone: mozilla0.9.2 → ---
giving up ancient string bugs to the new string owner. jag, you'll want to sort through these and see which ones still apply and go with or against the direction in which you intend strings evolve
Assignee: scc → jaggernaut
Status: ASSIGNED → NEW
This got fixed in some way or other a long time ago.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.