Closed Bug 171150 Opened 22 years ago Closed 9 years ago

Unify nsTXTToHTMLConv and mozTXTToHTMLConv

Categories

(Core :: Networking, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: BenB, Unassigned)

References

Details

(Keywords: helpwanted, memory-footprint)

mozTXTToHTMLConv was mostly written (by me) for Mailnews, but with other consumers in mind. Thus, it was intended to be a stream converter, but doesn't support the (full?) API yet. The class has quite fancy conversion functions. nsTXTToHTMLConv was written way after that by valeski when a URL recognizer for gopher was needed. It does URL recognition, but that is inferiour to the routines in moz*. valeski said that he added the ns* version, because the moz* one were not made for asyncronous use. While it is true that the latter doesn't support the async stream converter APIs, this can be implemented relatively easily (it's straightforward), it's just that nobody bothered to do it. My suggestions would be to do that and get rid of the ns* class. This is the state as I remember it. Fixing this bug would fix 68044.
ben - what are the chances you can add the async work here? Or, is there someone more familiar with this stuff that can outline what is necessary to make this async so that someone else can tackle this?
> what are the chances you can add the async work here? I have no motivation / incentive to work on this (- no more). You could contract me :-). However, I can support any effort to implement this. I can dump my thoughts on how I think this can be implemented, answer questions about how the class works, etc..
> I have no motivation / incentive to work on this (- no more). You could contract me :-). I'll give you US$6.50 - that's all I've got on me and I won't be able to do laundry this weekend.
haha. hmm, I'll pass, but thank you. I cannot implement this within 5 minutes and, more importantly, I cannot justify making you run around in dirty cloths - you could lose your job!
Blocks: 129472
geez
Assignee: nobody → nobody
No longer blocks: 129472
Depends on: 20212
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.