Closed Bug 339260 Opened 18 years ago Closed 18 years ago

ns_not_implemented should probably be ns_error_not_implemented

Categories

(Calendar :: Internal Components, defect)

Product:
Calendar
Component:
Internal Components
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: csthomas, Assigned: jminta)

References

(
URL
)

Details

Attachments

(1 file)

use NS_ERROR_NOT_IMPLEMENTED
1.72 KB, patch
dmosedale
: first-review+
Details | Diff | Splinter Review 
/calendar/base/src/calItemBase.js, line 443 -- throw Components.results.NS_NOT_IMPLEMENTED;
/calendar/base/src/calItemBase.js, line 447 -- throw Components.results.NS_NOT_IMPLEMENTED;
/calendar/base/src/calItemBase.js, line 609 -- throw Components.results.NS_NOT_IMPLEMENTED;
Attached patch use NS_ERROR_NOT_IMPLEMENTED — — Splinter Review 
I suppose it's a good thing we've never hit these errors.
Assignee: base → jminta
Status: NEW → ASSIGNED
Attachment #225801 - Flags: first-review?(dmose)
Comment on attachment 225801 [details] [diff] [review]
use NS_ERROR_NOT_IMPLEMENTED

r=dmose
Attachment #225801 - Flags: first-review?(dmose) → first-review+
patch checked in.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: