Closed
Bug 286876
Opened 20 years ago
Closed 19 years ago
In <phishingDetector.js>, "Warning: redeclaration of var index"
Categories
(Thunderbird :: Mail Window Front End, defect)
Thunderbird
Mail Window Front End
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird1.1
People
(Reporter: sgautherie, Assigned: sgautherie)
References
Details
Attachments
(1 file)
(deleted),
patch
|
mscott
:
review+
mscott
:
superreview+
asa
:
approval-aviary1.1a1-
asa
:
approval-aviary1.1a2+
|
Details | Diff | Splinter Review |
[Mozilla Thunderbird, version 1.0+ (20050319)] (nightly) (W98SE)
(not on v1.0 branch !?)
{{
Warning: redeclaration of var index
Source File: chrome://messenger/content/phishingDetector.js
Line: 149, Column: 11
Source Code:
for (var index = 0; index < ipComponents.length; index++)
}}
when starting TB.
Assignee | ||
Updated•20 years ago
|
Component: Security → Mail Window Front End
Depends on: mailphish
Target Milestone: --- → Thunderbird1.1
Assignee | ||
Comment 1•20 years ago
|
||
[Mozilla Thunderbird, version 1.0+ (20050319)] (nightly) (W98SE)
Could you review/check in this patch ? Thanks.
Updated•20 years ago
|
Attachment #177974 -
Flags: superreview+
Attachment #177974 -
Flags: review?(mscott)
Attachment #177974 -
Flags: review+
Assignee | ||
Updated•20 years ago
|
Whiteboard: [checkin-needed]
Assignee | ||
Updated•20 years ago
|
Attachment #177974 -
Flags: approval-aviary1.1a?
Updated•20 years ago
|
Attachment #177974 -
Flags: approval-aviary1.1a? → approval-aviary1.1a-
Assignee | ||
Comment 2•19 years ago
|
||
Comment on attachment 177974 [details] [diff] [review]
(Av1) <phishingDetector.js>
[Checked in: Comment 3]
'approval1.8a2=?':
Trivial JS code cleanup, no risk.
Attachment #177974 -
Flags: approval-aviary1.1a2?
Updated•19 years ago
|
Attachment #177974 -
Flags: approval-aviary1.1a2? → approval-aviary1.1a2+
Comment 3•19 years ago
|
||
checked in
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•19 years ago
|
Attachment #177974 -
Attachment description: (Av1) <phishingDetector.js> → (Av1) <phishingDetector.js>
[Checked in: Comment 3]
Attachment #177974 -
Attachment is obsolete: true
Assignee | ||
Updated•19 years ago
|
Whiteboard: [checkin-needed]
Updated•19 years ago
|
Attachment #177974 -
Attachment is obsolete: false
You need to log in
before you can comment on or make changes to this bug.
Description
•