Closed Bug 295765 Opened 19 years ago Closed 19 years ago

remove hand-rolled pop3 timeouts now that necko has timeouts

Categories

(MailNews Core :: Networking: POP, defect)

Product:
MailNews Core
Component:
Networking: POP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Details

Attachments

(1 file)

proposed fix
5.27 KB, patch
mscott
: superreview+
shaver
: approval-aviary1.1a2+
Details | Diff | Splinter Review
Attached patch proposed fix — — Splinter Review 
just remove the hand-rolled timeout code.
Attachment #184733 - Flags: superreview?(mscott)
Attachment #184733 - Flags: superreview?(mscott) → superreview+
Attachment #184733 - Flags: approval-aviary1.1a2?
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
thx, yeah, I'll do that in my tree.
Comment on attachment 184733 [details] [diff] [review]
proposed fix

a=shaver, woo!
Attachment #184733 - Flags: approval-aviary1.1a2? → approval-aviary1.1a2+
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: