Closed Bug 298624 Opened 19 years ago Closed 19 years ago

redeclaration of constant kMailToLength in nsContextMenu.js

Categories

(Thunderbird :: General, defect)

Product:
Thunderbird
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
Thunderbird1.1

People

(Reporter: whimboo, Assigned: whimboo)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

patch v1
2.60 KB, patch
mscott
: review+
Details | Diff | Splinter Review 
The file nsContextmenu.js is included by messenger.xul and messageWindow.xul. So
the constant is defined twice. Following error message occurs:

Error: redeclaration of const kMailToLength
Source File: chrome://communicator/content/nsContextMenu.js
Line: 13
Attached patch patch v1 — — Splinter Review 
Scott, I have reverted your changes of version 1.9. Now the constant is defined
within the function itself and no hardcoded values are used.
Assignee: mscott → hskupin
Status: NEW → ASSIGNED
Attachment #187154 - Flags: review?(mscott)
Attachment #187154 - Flags: review?(mscott) → review+
fixed. thanks for the patch.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird1.1
Status: RESOLVED → VERIFIED
Blocks: 303545
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: