fix(print): Fix format specifiers #12164
Conversation
lucasssvaz
requested review from
a team,
SuGlider and
me-no-dev
as code owners
👋 Hello lucasssvaz, we appreciate your contribution to this project! 📘 Please review the project's Contributions Guide for key guidelines on code, documentation, testing, and more. 🖊️ Please also make sure you have read and signed the Contributor License Agreement for this project. Click to see more instructions ...
Review and merge process you can expect ...
|
Memory usage test (comparing PR against master branch)The table below shows the summary of memory usage change (decrease - increase) in bytes and percentage for each target.
Click to expand the detailed deltas report [usage change in BYTES]
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Test Results 90 files 90 suites 31m 45s ⏱️ Results for commit e44cec4. |
1 participant
Description of Change
This pull request standardizes the usage of format specifiers in
Serial.printfcalls across multiple example files, replacing%dand%uwith%zu(forsize_tvalues) and%lu(forunsigned longvalues). This improves type safety and ensures correct output formatting, especially on platforms wheresize_tand related types may differ in size fromintorunsigned int.Serial output formatting improvements:
%dand%uwith%zuinSerial.printfstatements for variables of typesize_tin UDP, Bluetooth, BLE, and Serial examples, ensuring proper formatting for buffer sizes, data lengths, and counts. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17] [18] [19] [20] [21] [22] [23]%ldto%luinSerial.printfforunsigned longvariables, such as delay times, to match the variable type in deep sleep and progress reporting examples. [1] [2]These changes help prevent formatting errors and potential undefined behavior, making the code more robust and portable across platforms.