Closed Bug 392572 Opened 17 years ago Closed 15 years ago

Correct comments in calIDateTime.idl

Categories

(Calendar :: Internal Components, defect)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mschroeder, Assigned: mschroeder)

Details

Attachments

(2 files)

"First day of year" starts with "1" and "day of month" end interval should also include "30".
Attachment #277076 - Flags: review?(daniel.boelzle)
Comment on attachment 277076 [details] [diff] [review]
[checked in] Patch v1

While you are editing documentation in that file: there's more worth to be added (@param etc)...
Attachment #277076 - Flags: review?(daniel.boelzle) → review+
checked in, thanks for the patch! more welcome...
Martin, could you please provide an update on the status of this bug now that the patch's been checked in?
Gary: No update, but see comment#2: "more welcome...". I have to work on more important things before a release than answering question on bugs that are not relevant to 0.8.
Whitespace cleanup & thrown exception corrected.
Attachment #320982 - Flags: review?(daniel.boelzle)
Comment on attachment 320982 [details] [diff] [review]
[checked in] Patch2 v1

r=dbo
Attachment #320982 - Flags: review?(daniel.boelzle) → review+
Attachment #277076 - Attachment description: Patch v1 → [checked in] Patch v1
Attachment #320982 - Attachment description: Patch2 v1 → [checked in] Patch2 v1
Can this bug be resolved fixed? If not: What is missing?
(In reply to comment #7)
> Can this bug be resolved fixed? If not: What is missing?

Nit-picks? See http://hg.mozilla.org/comm-central/file/bd99c66e725e/calendar/base/public/calIDateTime.idl ...

I guess everyone's still busy... (See comment #4 too)
Okay, let's mark this fixed, and file a new bug later if needed.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.9
You need to log in before you can comment on or make changes to this bug.