Closed
Bug 171150
Opened 22 years ago
Closed 9 years ago
Unify nsTXTToHTMLConv and mozTXTToHTMLConv
Categories
(Core :: Networking, enhancement)
Core
Networking
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.
Comment 1•22 years ago
|
||
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?
Keywords: footprint,
helpwanted
Reporter | ||
Comment 2•22 years ago
|
||
> 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..
Comment 3•22 years ago
|
||
> 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.
Reporter | ||
Comment 4•22 years ago
|
||
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!
Updated•9 years ago
|
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.
Description
•