Backport #8101
closedBackport fix for time.send(:initialize) reinitializing and modifying value
Description
I filed https://bugs.ruby-lang.org/issues/8099 to get Time to prevent reinitialization, and nobu fixed it in r39766. I am requesting that it be backported to 1.9.3 along with tests I will be adding shortly.
Updated by headius (Charles Nutter) about 11 years ago
Nevermind about additional tests...I see nobu did add one. So I'd just like to backport that commit to 1.9.3.
Updated by usa (Usaku NAKAMURA) about 11 years ago
- Status changed from Open to Assigned
- Assignee set to usa (Usaku NAKAMURA)
Updated by usa (Usaku NAKAMURA) almost 11 years ago
- Project changed from Backport193 to Backport200
- Assignee changed from usa (Usaku NAKAMURA) to nagachika (Tomoyuki Chikanaga)
Seems to be backported to 2.0.0 first.
Updated by nagachika (Tomoyuki Chikanaga) almost 11 years ago
Sorry for my late response. I'll merge r39766 and r39769. r39769 delete broken test by denial of re-initialize Time.
Updated by nagachika (Tomoyuki Chikanaga) almost 11 years ago
- Status changed from Assigned to Closed
- % Done changed from 0 to 100
This issue was solved with changeset r40282.
Charles, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.
merge revision(s) 39766,39769: [Backport #8101]
* time.c (GetTimeval): check if already initialized instance.
* time.c (GetNewTimeval): check if newly created instance.
* time.c (time_init_0, time_init_1, time_init_copy, time_mload): must
be newly created instance. [ruby-core:53436] [Bug #8099]
Updated by nagachika (Tomoyuki Chikanaga) almost 11 years ago
- Project changed from Backport200 to Backport193
- Status changed from Closed to Assigned
- Assignee changed from nagachika (Tomoyuki Chikanaga) to usa (Usaku NAKAMURA)
Updated by usa (Usaku NAKAMURA) almost 11 years ago
- Status changed from Assigned to Closed
This issue was solved with changeset r40711.
Charles, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.
merge revision(s) 39766,39769: [Backport #8101]
* time.c (GetTimeval): check if already initialized instance.
* time.c (GetNewTimeval): check if newly created instance.
* time.c (time_init_0, time_init_1, time_init_copy, time_mload): must
be newly created instance. [ruby-core:53436] [Bug #8099]
Updated by ktsj (Kazuki Tsujimoto) over 10 years ago
- Project changed from Backport193 to Backport200
- Status changed from Closed to Assigned
- Assignee changed from usa (Usaku NAKAMURA) to nagachika (Tomoyuki Chikanaga)
r42456 is related to r39766, so please backport it, too.
Updated by nagachika (Tomoyuki Chikanaga) about 10 years ago
- Status changed from Assigned to Closed
Applied in changeset r44844.
merge revision(s) r42456: [Backport #8101]
* time.c (get_timeval, get_new_timeval): use rb_obj_class()
instead of CLASS_OF() because CLASS_OF() may return
a singleton class.