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

Use fixed size strings and containers, remove dynamic allocation #68

Closed
12 tasks done
ecourtois opened this issue Jun 3, 2022 · 0 comments · Fixed by #95
Closed
12 tasks done

Use fixed size strings and containers, remove dynamic allocation #68

ecourtois opened this issue Jun 3, 2022 · 0 comments · Fixed by #95
Assignees
@ecourtois
Labels
enhancement New feature or request

Comments

@ecourtois
Copy link
Contributor

ecourtois commented Jun 3, 2022 •
edited
Loading

Remove dynamic memory allocation to have a better control of memory usage.

This task will be split in sub-tasks for each modules:

Dynamic allocation

Fixed-size containers

  • lib/cogip_defs/Polygon
  • lib/obstacles
  • lib/path
  • lib/avoidance
  • applications (obstacles, samples, ...)
  • strings
@ecourtois ecourtois added the enhancement New feature or request label Jun 3, 2022
@ecourtois ecourtois self-assigned this Jun 3, 2022
@ecourtois ecourtois changed the title Use fixed size strings and arrays Use fixed size strings and containers, remove dynamic allocation Aug 16, 2022
@ecourtois ecourtois linked a pull request Aug 19, 2022 that will close this issue
Use fixed size strings and containers, remove dynamic allocation #95
Merged
ecourtois added a commit that referenced this issue Sep 25, 2022
@ecourtois
lib: wizard: fix message handler registration
b13eeb7 
cogip::uartpb::message_handler_t was converted from std::function to
etl::delegate in [1], but the registration of wizard message handler was not
updated accordingly.

[1] #68

Signed-off-by: Eric Courtois <eric.courtois@gmail.com>
ecourtois added a commit that referenced this issue Sep 25, 2022
@ecourtois
lib: wizard: fix message handler registration
7eb5844 
cogip::uartpb::message_handler_t was converted from std::function to
etl::delegate in #68,
but the registration of wizard message handler was not updated accordingly.

Signed-off-by: Eric Courtois <eric.courtois@gmail.com>
This was referenced Sep 25, 2022
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ecourtois ecourtois

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use fixed size strings and containers, remove dynamic allocation cogip/mcu-firmware
1 participant
@ecourtois