Closed
Bug 304507
Opened 20 years ago
Closed 20 years ago
Ignore matches fails for CASEMAPPINGS outside of A-Z
Categories
(Other Applications Graveyard :: ChatZilla, defect)
Other Applications Graveyard
ChatZilla
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Gijs, Assigned: bugzilla-mozilla-20000923)
References
()
Details
(Whiteboard: [cz-0.9.69])
Attachments
(1 file)
|
923 bytes,
patch
|
samuel
:
review+
Gijs
:
review+
|
Details | Diff | Splinter Review |
... which means it buggers because the ignores are lowercased using that method,
and /i doesn't cut it when a server uses rfc1459 charsets. Patch by Silver
coming up, as I understand it.
|
Assignee | |
Updated•20 years ago
|
Summary: Ignore matches using the /i regexp option instead of lowercasing using e.server.toLowerCase → Ignore matches fails for CASEMAPPINGS outside of A-Z
|
|
Assignee | |
Comment 1•20 years ago
|
||
Attachment #192549 -
Flags: review?(samuel)
|
Reporter | |
Comment 2•20 years ago
|
||
Comment on attachment 192549 [details] [diff] [review]
Lower-cases nickname properly before comparing
Discussed on IRC, seems fine to me :-).
Attachment #192549 -
Flags: review+
|
||
Updated•20 years ago
|
Attachment #192549 -
Flags: review?(samuel) → review+
|
||
Comment 3•20 years ago
|
||
shouldnt this actually be a bug in the regexp parsing engine?
|
|
Assignee | |
Comment 4•20 years ago
|
||
Um, why? The problem is ^, [, ] and \ are not being matched to ~, {, } and |.
|
|
Assignee | |
Comment 5•20 years ago
|
||
Checked in --> FIXED.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•20 years ago
|
Whiteboard: [cz-0.9.68.6] → [cz-0.9.69]
|
||
Updated•4 months ago
|
Product: Other Applications → Other Applications Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•