Warning: redeclaration of var categoryList

RESOLVED FIXED

Status

SeaMonkey
Search
RESOLVED FIXED
15 years ago
10 years ago

People

(Reporter: timeless, Assigned: Callek)

Tracking

Trunk
x86
Windows 2000

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 obsolete attachment)

(Reporter)

Description

15 years ago
Warning: redeclaration of var categoryList
Source File: chrome://communicator/content/search/search-editor.js
Line: 140, Column: 6
Source Code:
  var categoryList = document.getElementById( "categoryList" );
(Assignee)

Comment 1

14 years ago
Created attachment 156671 [details] [diff] [review]
Move declaration of CategoryList

Move declaration to begining of scope tree to ensure |var| is only used once
for the variable.

Timeless, please mark review where appropriate.
(Reporter)

Updated

14 years ago
Attachment #156671 - Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #156671 - Flags: review+

Updated

14 years ago
Attachment #156671 - Flags: superreview?(neil.parkwaycc.co.uk) → superreview+
(Reporter)

Updated

14 years ago
Assignee: timeless → jw6057
(Reporter)

Comment 2

14 years ago
Comment on attachment 156671 [details] [diff] [review]
Move declaration of CategoryList

mozilla/xpfe/components/search/resources/search-editor.js	1.32
Attachment #156671 - Attachment is obsolete: true
(Assignee)

Comment 3

14 years ago
I do wonder why the patch was marked obsolete, but from my understanding this is
fixed
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Product: Core → SeaMonkey
You need to log in before you can comment on or make changes to this bug.