Closed Bug 736438 Opened 12 years ago Closed 7 years ago

Remove nsIUnicodeDecoder::GetCharacterForUnMapped

Categories

(Core :: Internationalization, defect)

Product:
Core
Component:
Internationalization
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla56

People

(Reporter: Ms2ger, Assigned: hsivonen)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Whiteboard: [fixed by encoding_rs]) 

It's only used in nsScanner, so we should probably remove it once nsScanner is gone.
(In reply to Ms2ger from comment #0)
> It's only used in nsScanner, so we should probably remove it once nsScanner
> is gone.

You could just hard-code the REPLACEMENT CHARACTER and not wait for nsScanner to go away.
The decoder error is always REPLACEMENT CHARACTER per encoding spec, so GetCharacterForUnMapped would be redundant.
Blocks: encoding
Depends on: 593338
Blocks: 819869
Summary: Remove nsIUnicodeDecoder::GetCharacterForUnMapped? → Remove nsIUnicodeDecoder::GetCharacterForUnMapped
Depends on: encoding_rs
Fixed by bug 1261841.
Assignee: nobody → hsivonen
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [fixed by encoding_rs]
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.