Closed
Bug 296559
Opened 20 years ago
Closed 20 years ago
make calIDateTime.isDate = true set hours, minutes, seconds to 0
Categories
(Calendar :: Internal Components, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: pavlov, Assigned: pavlov)
Details
Attachments
(1 file)
|
1.31 KB,
patch
|
vlad
:
first-review+
|
Details | Diff | Splinter Review |
make calIDateTime.isDate = true set hours, minutes, seconds to 0
|
Assignee | |
Comment 1•20 years ago
|
||
Attachment #185285 -
Flags: first-review?(vladimir)
Comment on attachment 185285 [details] [diff] [review]
fix
> NS_IMETHODIMP
>+calDateTime::SetIsDate(PRBool aIsDate)
>+{
>+ if (mImmutable)
>+ return NS_ERROR_FAILURE;
>+
>+ mIsDate = aIsDate;
>+ if (aIsDate) {
>+ mHour = 0;
>+ mMinute = 0;
>+ mSecond = 0;
Should Normalize here to make sure mNativeTime gets updated
>+ }
>+ return NS_OK;
>+}
with that, r=me
Attachment #185285 -
Flags: first-review?(vladimir) → first-review+
|
|
Assignee | |
Comment 3•20 years ago
|
||
checked in
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•