-
-
Notifications
You must be signed in to change notification settings - Fork 39.4k
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
Improve string sending #376
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
eltang
commented
Jun 3, 2016
- Move strings and LUTs to flash in order to reduce RAM consumption
- Rename LUTs for clarity
- Split lines longer than 80 characters
This check stuff is fancy. |
Cool, thanks. Let's skip the reformatting (line splitting) stuff in the future - it's just pedantic and not needed. Also, renaming should really be up to the collaborators. |
All right, but didn't you say you liked code to be readable? |
ryaninvents
pushed a commit
to ryaninvents/qmk_firmware
that referenced
this pull request
Aug 12, 2016
jiaxin96
pushed a commit
to Oh-My-Mechanical-Keyboard/qmk_firmware
that referenced
this pull request
Oct 18, 2023
KarlK90
added a commit
to KarlK90/qmk_firmware
that referenced
this pull request
Nov 28, 2023
This contains commit from da78eb3759b8..9d7a7f904ed1: 9d7a7f90 Merge pull request qmk#382 from KarlK90/fix/rp2040-i2c-speeds 70119934 Merge pull request qmk#383 from KarlK90/fix/rp2040-usb-get-status-request 1a1bbe6c rp2040: usb: fix usb_lld_get_status functions 7d9212dd rp2040: i2c: fix speed calculation fb67e502 Merge pull request qmk#377 from 1Conan/sn32_fix_registry e72939ef SN32: update registry 5b4836ca Merge pull request qmk#376 from dexter93/sn32_usb_v2 5ded9de9 sn32: usb: do NOT clear interrupt status until handled Leftover Sonix reference code cleanup. Sometimes when there is traffic on more than 1 ep's packets would be dropped before they could be handled. Clearing the status flags after handling them takes care of it. e9a4a512 sn32: usb: only activate interrupts on lld start e4a35d1c sn32: fix host remote wakeup
KarlK90
added a commit
that referenced
this pull request
Nov 29, 2023
This contains commit from da78eb3759b8..9d7a7f904ed1: 9d7a7f90 Merge pull request #382 from KarlK90/fix/rp2040-i2c-speeds 70119934 Merge pull request #383 from KarlK90/fix/rp2040-usb-get-status-request 1a1bbe6c rp2040: usb: fix usb_lld_get_status functions 7d9212dd rp2040: i2c: fix speed calculation fb67e502 Merge pull request #377 from 1Conan/sn32_fix_registry e72939ef SN32: update registry 5b4836ca Merge pull request #376 from dexter93/sn32_usb_v2 5ded9de9 sn32: usb: do NOT clear interrupt status until handled Leftover Sonix reference code cleanup. Sometimes when there is traffic on more than 1 ep's packets would be dropped before they could be handled. Clearing the status flags after handling them takes care of it. e9a4a512 sn32: usb: only activate interrupts on lld start e4a35d1c sn32: fix host remote wakeup
leotaku
pushed a commit
to leotaku/qmk_firmware
that referenced
this pull request
Dec 1, 2023
nuess0r
pushed a commit
to nuess0r/qmk_firmware
that referenced
this pull request
Sep 8, 2024
This contains commit from da78eb3759b8..9d7a7f904ed1: 9d7a7f90 Merge pull request qmk#382 from KarlK90/fix/rp2040-i2c-speeds 70119934 Merge pull request qmk#383 from KarlK90/fix/rp2040-usb-get-status-request 1a1bbe6c rp2040: usb: fix usb_lld_get_status functions 7d9212dd rp2040: i2c: fix speed calculation fb67e502 Merge pull request qmk#377 from 1Conan/sn32_fix_registry e72939ef SN32: update registry 5b4836ca Merge pull request qmk#376 from dexter93/sn32_usb_v2 5ded9de9 sn32: usb: do NOT clear interrupt status until handled Leftover Sonix reference code cleanup. Sometimes when there is traffic on more than 1 ep's packets would be dropped before they could be handled. Clearing the status flags after handling them takes care of it. e9a4a512 sn32: usb: only activate interrupts on lld start e4a35d1c sn32: fix host remote wakeup
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.