bad assumption that time_t is 32b
time_t can be 64 bits but in NeL we suppose that time_t is 32b and there's code that convert or store time_t under a uint32.
uint32 CTime::getSecondsSince1970 ()
The function must returns a time_t and check all linked issues.
To test, you can returning a uint64 and it'll generate warning where problems occur
#1 Updated by abelgar over 4 years ago
- File Feature__33.patch added
Here, a small patch which corrects the bad assumption. All linked issues has been corrected.
#2 Updated by abelgar over 4 years ago
- File Feature__33_v2.patch added
clean version with add of an omission
#3 Updated by sfb about 4 years ago
- Category set to Misc
- Assignee set to sfb
- Target version set to Version 0.7.0
#4 Updated by sfb over 3 years ago
- Priority changed from Normal to High
- Target version changed from Version 0.7.0 to 0.8.0
So this is going to take some focus and testing, I'm changing the priority and moving it to 0.8.0 with hopes that it'll get fixed in trunk early in the 0.8.0 development cycle.