Skip to content

arch/arm/src/stm32h7/stm32_usbhost.c: Fix nxstyle errors#9280

Merged
xiaoxiang781216 merged 1 commit into
apache:masterfrom
simbit18:simbit18-20230515
May 18, 2023
Merged

arch/arm/src/stm32h7/stm32_usbhost.c: Fix nxstyle errors#9280
xiaoxiang781216 merged 1 commit into
apache:masterfrom
simbit18:simbit18-20230515

Conversation

@simbit18
Copy link
Copy Markdown
Contributor

Summary

error: Long line found

Impact

none

Testing

@davids5
Copy link
Copy Markdown
Contributor

davids5 commented May 15, 2023

@simbit18

Are you aware that the initial comment line sets the the "Ruler" and then lines can be longer?

This change makes the file less readable. I would not support merging it.

@simbit18
Copy link
Copy Markdown
Contributor Author

simbit18 commented May 15, 2023

@xiaoxiang781216
Copy link
Copy Markdown
Contributor

@davids5 only some long syslog split into two lines, the result isn't too bad, I think.

@xiaoxiang781216 xiaoxiang781216 merged commit 0b97979 into apache:master May 18, 2023
@simbit18 simbit18 deleted the simbit18-20230515 branch May 18, 2023 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants