Closed Bug 1717367 Opened 4 years ago Closed 4 years ago

Remove MAX_LINES_PER_SEND from cZ

Categories

(SeaMonkey :: Chat, task)

Tracking

(seamonkey2.53+ fixed)

RESOLVED FIXED
seamonkey 2.88
Tracking Status
seamonkey2.53 + fixed

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

References

Details

(Whiteboard: SM2.53.9)

Attachments

(1 file)

As mentioned over in Ambassador in issue #80:
Simplify the code by removing the MAX_LINES_PER_SEND property/setting, it has been defaulted to 0 (unlimited) for years. With the way it's implemented now there isn't even a user-visible way of telling whether that limit was reached without relying on echo-message. Most servers moderate this sort of thing anyway, so it's just best to remove it.
Cherry picking the fix from Ambassador:
https://github.com/Ascrod/ambassador/commit/600bd0e9c6d3faf9930c5f5f1c4416ecfb271580

[Triage Comment]
r/a=me

Attachment #9228020 - Flags: review+
Attachment #9228020 - Flags: approval-comm-release+
Attachment #9228020 - Flags: approval-comm-esr60+
Blocks: 1717369

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/b2d3379e2cb8
Remove MAX_LINES_PER_SEND from cZ. r=IanN

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Whiteboard: SM2.53.9
Target Milestone: --- → seamonkey 2.88
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: