Closed Bug 297796 Opened 20 years ago Closed 19 years ago

Make /cancel abort/stop a running /list

Categories

(Other Applications :: ChatZilla, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bugzilla-mozilla-20000923, Assigned: Gijs)

Details

(Whiteboard: [cz-0.9.69])

Attachments

(1 file)

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.
Attached patch PatchSplinter Review
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)
Assignee: silver → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Attachment #204244 - Flags: review?(samuel) → review+
Checked in --> FIXED.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Whiteboard: [cz-0.9.69]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: