MDot 4.0.1 library compilation warning in mDotEvent.h

[Ajay K]

I see this warning when compiling in mDotEvent.h in mdot 4.0.1 library and was wondering if this needs to be fixed as the read_ms() is deprecated in mbed os 6.1.0 or is it okay to ignore this warning?

[Warning] mDotEvent.h@279,64: ‘int mbed::TimerBase::read_ms() const’ is deprecated: Use the Chrono-based elapsed_time method. If integer milliseconds are needed, you can use duration_cast<milliseconds>(elapsed_time()).count() [since mbed-os-6.0.0] [-Wdeprecated-declarations]

Thanks,
Ajay

[Jason Reiss]

It is a warning that this may not be supported in a future release but still appears to work as is.

[Ajay K]

Thanks Jason for taking the time to respond, however i see it says read_ms is deprecated since mbed-os 6.0.0 and the version that is being used mdot library is mbed os 6.1.0 and when I look at mdotevent.h, all the lines of code above that warning seems to have been updated to use the chrono based elapsed_time, except for this line of code where the warning is generated. Is this intentional or something that was missed?

Thanks,
Ajay

[Ajay K]

Hey Jason,

I think like you say it works as is currently and you don’t foresee any issues right based on my earlier comments? I just don’t want to deploy this in production and later find out we run into issues.

Thanks,
Ajay

[Jason Reiss]

How about this, the read_ms function will return milliseconds time until it is removed from the mbed-os API. At that point this code will not compile. That is why it is marked deprecated.

[Author]

Posts