Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

examples/usbserial: Fix style issues in host.c #2940

Merged
merged 1 commit into from
Jan 11, 2025

Conversation

no1wudi
Copy link
Contributor

@no1wudi no1wudi commented Jan 11, 2025

Summary

Fix code style issue found in #2939

  • Break long preprocessor conditionals across multiple lines
  • Add spaces around operators for better readability
  • Reformat long string literals to fit within line length limits
  • Improve consistency in spacing around parentheses and brackets

Impact

Code style only

Testing

GitHub CI

@nuttxpr
Copy link

nuttxpr commented Jan 11, 2025

[Experimental Bot, please feedback here]

Yes. While the PR lacks specific build/target info, it clearly addresses a code style concern stemming from another PR, making the context clear. The impact is explicitly limited to code style, and testing relies on GitHub CI, which is acceptable for such a change. It would be improved by adding specific build host and target information, even if just confirming that it built cleanly on the developer's local system.

@no1wudi no1wudi force-pushed the clean branch 2 times, most recently from 9adf470 to de858f0 Compare January 11, 2025 01:17
- Break long preprocessor conditionals across multiple lines
- Add spaces around operators for better readability
- Reformat long string literals to fit within line length limits
- Improve consistency in spacing around parentheses and brackets

Signed-off-by: Huang Qi <[email protected]>
Copy link

@cederom cederom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @no1wudi :-)

@xiaoxiang781216 xiaoxiang781216 merged commit 46f77a6 into apache:master Jan 11, 2025
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants