lljwork2021 opened a new pull request, #20103:
URL: https://github.com/apache/nuttx/pull/20103

   Allow public eProsima Micro XRCE-DDS Client and Micro-CDR type names
   used by the XRCE-DDS example.
   
   ## Summary
   
   The XRCE-DDS example uses public type names from the eProsima
   Micro XRCE-DDS Client and Micro-CDR APIs. Some of these identifiers
   use mixed-case naming and cannot be renamed locally without changing
   the upstream API names.
   
   Add these identifiers to the nxstyle whitelist so that the XRCE-DDS
   example can pass the NuttX style checks while preserving the upstream
   API names.
   
   Related to 
[apache/nuttx-apps#3777](https://github.com/apache/nuttx-apps/pull/3777).
   
   ## Impact
   
   This change only extends the nxstyle identifier whitelist.
   
   There is no runtime, ABI, hardware, or functional impact.
   
   ## Testing
   
   Verified the XRCE-DDS source files with NuttX style checks after
   adding the required public API identifiers to the whitelist.
   
   The previous `Mixed case identifier found` errors for these
   Micro XRCE-DDS / Micro-CDR identifiers are no longer reported.
   
   ```sh
   ~/nuttxspace/nuttx$ ./tools/checkpatch.sh -c -m -g upstream/master..HEAD
   Used config files:
       1: .codespellrc
   ✔️ All checks pass.
   ```


-- 
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]

Reply via email to