feat(update): Add Doxygen style documentation for the Update class#12243
feat(update): Add Doxygen style documentation for the Update class#12243
Conversation
|
@lbernstone does this fulfill your requirement? |
👋 Hello me-no-dev, 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 ...
|
Test Results 90 files 90 suites 32m 47s ⏱️ Results for commit c813e84. ♻️ This comment has been updated with latest results. |
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]
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Looks good. There's still a couple /* remarked functions at the end, and the helper functions are empty, but fairly self-explanatory. |
This pull request enhances the documentation and clarity of the
UpdateClassAPI inUpdate.hby replacing terse or outdated comments with detailed Doxygen-style documentation. It also improves the readability of macro definitions by adding descriptive comments for each constant. No functional or behavioral changes have been made to the code itself.The most important changes are:
API Documentation Improvements:
UpdateClasswith detailed Doxygen-style documentation, describing parameters, return values, and method behavior.Macro and Constant Clarification:
UPDATE_ERROR_*,U_FLASH,ENCRYPTED_BLOCK_SIZE) to clarify their purpose and usage throughout the file.Fixes: #12241