Closed Bug 125755 Opened 23 years ago Closed 4 years ago

Add an option to GetCharsetAtom to avoid charset alias resolution

Categories

(Core :: Internationalization, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: nhottanscp, Unassigned)

References

Details

(Keywords: intl, perf)

http://lxr.mozilla.org/seamonkey/source/intl/uconv/idl/nsICharsetConverterManager2.idl#109 nsICharsetConverterManager2::GetCharsetAtom takes a string and convert it to atom. The implementaion uses the charset alias manager which is not inexpensive. The function is needed everytime creating decoder/encoder, so we want to improve the performance there. The alias resolution is not needed when the claller knows the standard name like "UTF-8" (case need to match). By adding an option to skip the alias resoultion, we can improve the performance.
Blocks: 115288
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.2
Keywords: intl
Keywords: mozilla1.0+
Target Milestone: mozilla1.2alpha → ---
QA Contact: ruixu → i18n
Assignee: nhottanscp → nobody
Status: ASSIGNED → NEW
Keywords: perf

mozilla::Encoding* has this role now.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.