Make seconds of Time_t 64bit #4425
Conversation
* dds/DCPS/Time_Helper.inl:
* dds/DdsDcpsCore.idl:
* dds/DdsDcpsInfoUtils.idl:
|
This will break the interoperability of applications that use |
|
This could break portability, interoperability is handled by RTPS and there is another different definition on time. The risk is very small, all Time_t is created by the OpenDDS core and passed to user code where they can use it. At the moment they use Time_t as type nothing will happen, the only risk is when they use another 32bit type to store it, at that moment they would get a warning, easily fixed by changing to 64bit also or add a cast. This is a break in portability we should do asap to my idea, not wait because we maybe cause a warning somewhere for just a few users. |
|
The risk is for applications that use |
|
We have added this to the roadmap under OpenDDS 4.0. |
Fixed #4365