Closed Bug 238080 Opened 20 years ago Closed 20 years ago

In <CookieViewer.js>, "Warning: redeclaration of var p"

Categories

(Core :: Networking: Cookies, defect)

defect
Not set
minor

Tracking

()

VERIFIED FIXED
mozilla1.7final

People

(Reporter: sgautherie, Assigned: sgautherie)

Details

Attachments

(1 obsolete file)

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.7b) Gecko/20040316] (W98SE)

Spun off from bug 233910 comment 7. (already in v1.7a)

{{
Warning: redeclaration of var p
Source File: chrome://communicator/content/wallet/CookieViewer.js
Line: 619, Column: 13
Source Code:
    for (var p=0; p<deletedPermissions.length; p++)
}}

Updating:
*(A.To) darin#meer.net -> me.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.8alpha
Attachment #144399 - Flags: superreview?(jag)
Attachment #144399 - Flags: review?(mvl)
Attachment #144399 - Flags: review?(mvl) → review+
Comment on attachment 144399 [details] [diff] [review]
(Av1) <CookieViewer.js>
[Checked in: Comment 5]

sr=jag
Attachment #144399 - Flags: superreview?(jag) → superreview+
Comment on attachment 144399 [details] [diff] [review]
(Av1) <CookieViewer.js>
[Checked in: Comment 5]


'approval1.7=?':
Trivial code fix, no risk;
(Trying v1.7 instead of waiting for v1.8a, as v1.7 will become new
long-lived...)
Attachment #144399 - Flags: approval1.7?
Comment on attachment 144399 [details] [diff] [review]
(Av1) <CookieViewer.js>
[Checked in: Comment 5]

a=mkaply
Attachment #144399 - Flags: approval1.7? → approval1.7+
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Target Milestone: mozilla1.8alpha → mozilla1.7final
Attachment #144399 - Attachment description: (Av1) <CookieViewer.js> → (Av1) <CookieViewer.js> [Checked in: Comment 5]
Attachment #144399 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: