Closed
Bug 297796
Opened 20 years ago
Closed 19 years ago
Make /cancel abort/stop a running /list
Categories
(Other Applications :: ChatZilla, defect)
Other Applications
ChatZilla
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bugzilla-mozilla-20000923, Assigned: Gijs)
Details
(Whiteboard: [cz-0.9.69])
Attachments
(1 file)
|
5.87 KB,
patch
|
samuel
:
review+
|
Details | Diff | Splinter Review |
We've all done it once: /list on quakenet, or efnet. Minutes later and it is still going... The "/eval e.network._list = []" stopps it quickly, though probably with a JS error or two. Need to add code into cmdCancel to detect a connected server with a /list running, and abort it gracefully.
| Assignee | ||
Comment 1•19 years ago
|
||
Patch. We add an extra flag to the list object, and look for that when displaying and when adding items. We no-op for all remaining items, and display the normal stopping message when done. This will usually list something like 'Displayed 30 out of 20542 channels' on things like quakenet. It will still take quite a while, but your client should be somewhat usable in the maintime. We can't stop ourselve trying to cope with the influx of LIST replies, nor can we stop the server, so we solve it this way.
Attachment #204244 -
Flags: review?(samuel)
| Reporter | ||
Updated•19 years ago
|
Assignee: silver → gijskruitbosch+bugs
| Assignee | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Updated•19 years ago
|
Attachment #204244 -
Flags: review?(samuel) → review+
| Reporter | ||
Comment 2•19 years ago
|
||
Checked in --> FIXED.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
| Reporter | ||
Updated•19 years ago
|
Whiteboard: [cz-0.9.69]
You need to log in
before you can comment on or make changes to this bug.
Description
•