Closed Bug 366141 Opened 19 years ago Closed 19 years ago

JavaScript strict warnings in migration.js and calendarManagement.js

Categories

(Calendar :: Internal Components, defect)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ssitter, Assigned: ssitter)

Details

Attachments

(1 file)

Attached patch fix warningsSplinter Review
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a2pre) Gecko/20070106 Calendar/0.4a1 Call me pedantic but I'm trained to write warning free code ... Warning: assignment to undeclared variable item Source File: chrome://calendar/content/calendarManagement.js Line: 41 Warning: assignment to undeclared variable localFile Source File: chrome://calendar/content/migration.js Line: 665 Warning: assignment to undeclared variable localFile Source File: chrome://calendar/content/migration.js Line: 712
Attachment #250695 - Flags: first-review?(jminta)
Status: NEW → ASSIGNED
Comment on attachment 250695 [details] [diff] [review] fix warnings r=jminta. nice job.
Attachment #250695 - Flags: second-review+
Attachment #250695 - Flags: first-review?(jminta)
Attachment #250695 - Flags: first-review+
Whiteboard: [needs checkin]
Checked in on MOZILLA_1_8_BRANCH and trunk. -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Whiteboard: [needs checkin]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: