espressif/openthread
1
0
Fork 0
mirror of https://github.com/espressif/openthread.git synced 2026-08-23 18:59:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

[logging] fix print format when enabling csl debug (#10045)

Browse Source 
Fix -Wformat trigger on some platforms as an int32_t is printed using %d

    ```
    src/core/mac/sub_mac.cpp:336:73: warning: format '%d' expects argument of type 'int', but argument 5 has type 'int32_t' {aka 'long int'} [-Wformat=]
      336 |     logString.Append("Expected sample time %lu, margin ±%lu, deviation %d", ToUlong(sampleTime), ToUlong(ahead),
          |                                                                        ~^
          |                                                                         |
          |                                                                         int
          |                                                                        %ld
      337 |                      deviation);
          |                      ~~~~~~~~~
          |                      |
          |                      int32_t {aka long int}
    ```
This commit is contained in:
Suvesh Pratapa
2024-04-18 15:49:38 -07:00
committed by GitHub
parent 2199db8251
commit 4c96151c0e
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/core/mac/sub_mac.cpp
+2 -2
View File
@@ -333,8 +333,8 @@ void SubMac::LogReceived(RxFrame *aFrame)
// be due to clocks drift and/or CSL Phase rounding error.
// This means that a deviation absolute value greater than the margin would result in the frame
// not being received out of the debug mode.
logString.Append("Expected sample time %lu, margin ±%lu, deviation %d", ToUlong(sampleTime), ToUlong(ahead),
deviation);
logString.Append("Expected sample time %lu, margin ±%lu, deviation %ld", ToUlong(sampleTime), ToUlong(ahead),
static_cast<long>(deviation));
// Treat as a warning when the deviation is not within the margins. Neither kCslReceiveTimeAhead
// or kMinReceiveOnAhead/kMinReceiveOnAfter are considered for the margin since they have no