You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by "ThePassionate (via GitHub)" <gi...@apache.org> on 2026/01/19 02:27:11 UTC

[PR] crypto/cryptosoft: replace macro howmany with common macro div_round_up [nuttx]

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 <ma...@xiaomi.com>


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] crypto/cryptosoft: replace macro howmany with common macro div_round_up [nuttx]

Posted by "GUIDINGLI (via GitHub)" <gi...@apache.org>.
GUIDINGLI merged PR #17994:
URL: https://github.com/apache/nuttx/pull/17994


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org