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.

Sign up for GitHub

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

Jump to bottom

Refactor messaging parsing: Use single-threaded state machine in place of two-threaded implementation #28

Closed
TheSomeMan opened this issue Oct 17, 2023 · 0 comments
Closed

Refactor messaging parsing: Use single-threaded state machine in place of two-threaded implementation #28

TheSomeMan opened this issue Oct 17, 2023 · 0 comments
Assignees
@ojousima

Comments

@TheSomeMan
Copy link
Collaborator

The change is made for multiple reasons:

  1. Performance Improvements.
  2. Simpler debugging due to the reduction of synchronization issues.
  3. Increased maintainability.
  4. Memory efficiency: This implementation contributes towards saving approximately 6.5 kB of memory.
@TheSomeMan TheSomeMan self-assigned this Oct 17, 2023
TheSomeMan added a commit that referenced this issue Oct 17, 2023
@TheSomeMan
[#28] Add googletest
54d718d
TheSomeMan added a commit that referenced this issue Oct 17, 2023
@TheSomeMan
[#28] Fix bug in macro print_dbgmsg
00d0b75
TheSomeMan added a commit that referenced this issue Oct 17, 2023
@TheSomeMan
[#28] Log ACK confirmation only if DEBUG mode enabled
026664b
TheSomeMan added a commit that referenced this issue Oct 17, 2023
@TheSomeMan
[#28] api_report_callback: print dbgmsg if message decoding failed
5cdd4db
TheSomeMan added a commit that referenced this issue Oct 17, 2023
@TheSomeMan
[#28] Fix incorrect callback ptr checking in 'parse'
b11d879
TheSomeMan added a commit that referenced this issue Oct 17, 2023
@TheSomeMan
Close #28: Refactor messaging parsing: Use single-threaded state mach…
426b36e 
…ine in place of two-threaded implementation
TheSomeMan added a commit that referenced this issue Oct 17, 2023
@TheSomeMan
[#28] Add unit-tests
a05488f
TheSomeMan added a commit that referenced this issue Oct 17, 2023
@TheSomeMan
[#28] Add GitHub action 'build'
d7b253f
TheSomeMan added a commit that referenced this issue Oct 17, 2023
@TheSomeMan
[#28] Fix GitHub actions
2acdbea
TheSomeMan added a commit that referenced this issue Oct 17, 2023
@TheSomeMan
[#28] Fix GitHub action 'build'
bc2fbd4
TheSomeMan added a commit that referenced this issue Oct 17, 2023
@TheSomeMan
[#28] Fix GitHub action 'build'
89b1301
@TheSomeMan TheSomeMan mentioned this issue Oct 17, 2023
Closed
TheSomeMan added a commit that referenced this issue Oct 25, 2023
@TheSomeMan
[#28] Increase the stack size for "uart_rx_task" from 2 to 4kB
da79603
TheSomeMan added a commit that referenced this issue Oct 25, 2023
@TheSomeMan
[#28] Increase the stack size for "uart_rx_task" from 2 to 4kB
baaafa4
@TheSomeMan TheSomeMan assigned ojousima and unassigned TheSomeMan Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ojousima ojousima

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ojousima @TheSomeMan