ThePassionate opened a new pull request, #17994:
URL: https://github.com/apache/nuttx/pull/17994
## Crypto Soft: Replace howmany with div_round_up
### Summary
This PR replaces the non-standard `howmany` macro with the common
`div_round_up` macro in the cryptosoft implementation. This improves code
consistency across the NuttX codebase and enhances maintainability.
### Changes
#### Files Modified
1. **crypto/cryptosoft.c**
- Replace `howmany` macro calls with standard `div_round_up` macro
- Improves code consistency with NuttX standards
### Technical Details
**Macro Replacement:**
- The `howmany` macro is a non-standard implementation for division with
rounding up
- `div_round_up` is the common NuttX standard macro for the same operation
- This change maintains identical functionality while improving consistency
**Benefits:**
- Aligns with NuttX coding standards
- Reduces use of non-standard macros
- Improves code readability and maintainability
- Facilitates future refactoring and maintenance
### Impact
- **Consistency**: Improves code consistency across the crypto subsystem
- **Maintainability**: Easier to maintain and understand
- **Standards**: Follows NuttX standard patterns
- **Compatibility**: No functional impact; maintains existing behavior
### Testing
No functional changes, existing tests continue to pass.
---
**Author**: makejian <[email protected]>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
