Closed Bug 392827 Opened 17 years ago Closed 17 years ago

Duplicated function in calendar-management.js

Categories

(Calendar :: General, defect)

defect
Not set
trivial

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: mschroeder, Assigned: mschroeder)

Details

Attachments

(1 file)

Attached patch Patch v1 — — Splinter Review
In Error Console with javascript.option.strict set:
Warning: redeclaration of property isSeparator
Source File: chrome://calendar/content/calendar-management.js
Line: 374

We have a duplicated function:
/calendar/base/content/calendar-management.js,
    * line 366 -- isSeparator: function cLTV_isSeparator(aRow) {
    * line 374 -- isSeparator: function cLTV_isSeparator(aRow) {
Attachment #277330 - Flags: review?(philipp)
Comment on attachment 277330 [details] [diff] [review]
Patch v1

Looks fine, thanks for spotting this!

Checked in on HEAD and MOZILLA_1_8_BRANCH

-> FIXED
Attachment #277330 - Flags: review?(philipp) → review+
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.7
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: