xiaoxiang781216 opened a new pull request, #19231: URL: https://github.com/apache/nuttx/pull/19231
## Summary `O_RDOK` and `O_WROK` are non-standard aliases for `O_RDONLY` and `O_WRONLY` respectively. Having two names for the same flag creates confusion, especially when aligning the flag values with Linux. This PR removes the aliases from `include/fcntl.h` and replaces all in-tree uses with the standard `O_RDONLY`/`O_WRONLY`. ## Impact No functional change. `O_RDOK` was defined as `O_RDONLY` and `O_WROK` as `O_WRONLY`, so the replacement is a pure text substitution that preserves existing behavior while using the standard POSIX flag names. A companion change for the apps repository (replacing the remaining `O_RDOK` usages there) is submitted separately as apache/nuttx-apps#3566. ## Testing - `tools/checkpatch.sh -c -u -m -g` (commit mode, same as CI) passes with no errors. - `git grep 'O_RDOK\|O_WROK'` confirms no remaining references in the tree. - Builds cleanly on affected configs. -- 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]
