Closed
Bug 295765
Opened 20 years ago
Closed 20 years ago
remove hand-rolled pop3 timeouts now that necko has timeouts
Categories
(MailNews Core :: Networking: POP, defect)
MailNews Core
Networking: POP
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Bienvenu, Assigned: Bienvenu)
Details
Attachments
(1 file)
|
5.27 KB,
patch
|
mscott
:
superreview+
shaver
:
approval-aviary1.1a2+
|
Details | Diff | Splinter Review |
|
Assignee | |
Comment 1•20 years ago
|
||
just remove the hand-rolled timeout code.
|
|
Assignee | |
Updated•20 years ago
|
Attachment #184733 -
Flags: superreview?(mscott)
|
|
||
Updated•20 years ago
|
Attachment #184733 -
Flags: superreview?(mscott) → superreview+
|
|
Assignee | |
Updated•20 years ago
|
Attachment #184733 -
Flags: approval-aviary1.1a2?
|
||
Comment 2•20 years ago
|
||
shouldn't the mailnews.js entry for the pref also be removed?
/mailnews/mailnews.js, line 116 -- pref("mail.pop3_response_timeout", 45); // in
seconds
OS: Windows XP → All
Hardware: PC → All
|
|
Assignee | |
Comment 3•20 years ago
|
||
thx, yeah, I'll do that in my tree.
|
||
Comment 4•20 years ago
|
||
Comment on attachment 184733 [details] [diff] [review]
proposed fix
a=shaver, woo!
Attachment #184733 -
Flags: approval-aviary1.1a2? → approval-aviary1.1a2+
|
|
Assignee | |
Updated•20 years ago
|
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
|
||
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•