Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Feature/rmaker init deinit (MEGH-6105)#341

Open
Asc91 wants to merge 3 commits into
espressif:masterespressif/esp-rainmaker:masterfrom
Asc91:feature/rmaker_init_deinitAsc91/esp-rainmaker:feature/rmaker_init_deinitCopy head branch name to clipboard
Open

Feature/rmaker init deinit (MEGH-6105)#341
Asc91 wants to merge 3 commits into
espressif:masterespressif/esp-rainmaker:masterfrom
Asc91:feature/rmaker_init_deinitAsc91/esp-rainmaker:feature/rmaker_init_deinitCopy head branch name to clipboard

Conversation

@Asc91

@Asc91 Asc91 commented Nov 8, 2024

Copy link
Copy Markdown

Description

  1. Create update rmaker_stop and rmaker_end to exit gracefully.
  2. Add unit test for esp_rmaker to check for potential memory leaks and avoid breaking changes.
  3. Fix memory leaks found using unit test.

Testing

Commits were tested with esp32s3, latest esp-rainmaker, ESP-IDF 5.1 and ESP-IDF master

Related Issues

Closes #279

@vikramdattu @shahpiyushv

@CLAassistant

CLAassistant commented Nov 8, 2024

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions Bot changed the title Feature/rmaker init deinit Feature/rmaker init deinit (MEGH-6105) Nov 8, 2024
@Asc91
Asc91 force-pushed the feature/rmaker_init_deinit branch from 88486fe to fa35ac1 Compare November 8, 2024 08:48
@Asc91
Asc91 force-pushed the feature/rmaker_init_deinit branch from fa35ac1 to 2348835 Compare November 8, 2024 08:53
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.

Rainmaker doesn't stop/ deinit (MEGH-4882)

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.