Skip to content

Furi, FuriHal: remove FreeRTOS headers leaks #3179

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
Merged
hedger merged 11 commits into from
Nov 1, 2023
Merged

Furi, FuriHal: remove FreeRTOS headers leaks #3179

hedger merged 11 commits into from
Nov 1, 2023

Conversation

@skotopes
Copy link
Contributor

@skotopes skotopes commented Oct 31, 2023 •
edited by hedger
Loading

What's new

  • Furi, FuriHal: remove FreeRTOS headers leaks

Verification

  • All apps touched by this PR

Checklist (For Reviewer)

  • PR has description of feature/bug or link to Confluence/Jira task
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix

@github-actions
Copy link

github-actions bot commented Oct 31, 2023 •
edited
Loading

Compiled f7 firmware for commit 72e9b429:

@github-actions
Copy link

github-actions bot commented Oct 31, 2023

PVS-Studio report for commit df7321ea:

@skotopes skotopes changed the title Furi: remove direct FreeRTOS timers use Furi, FuriHal: remove FreeRTOS header leaks Oct 31, 2023
DrZlo13
DrZlo13 previously approved these changes Oct 31, 2023
View reviewed changes
@hedger hedger added Core+Services HAL, furi & core system services App Loader FAP loader + API-related labels Oct 31, 2023
@skotopes skotopes changed the title Furi, FuriHal: remove FreeRTOS header leaks Furi, FuriHal: remove FreeRTOS headers leaks Oct 31, 2023
@hedger hedger merged commit aa06328 into dev Nov 1, 2023
@hedger hedger deleted the aku/timers branch November 1, 2023 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hedger hedger hedger approved these changes

@DrZlo13 DrZlo13 DrZlo13 left review comments

@gsurkov gsurkov Awaiting requested review from gsurkov gsurkov is a code owner

@nminaylov nminaylov Awaiting requested review from nminaylov nminaylov is a code owner

@gornekich gornekich Awaiting requested review from gornekich gornekich is a code owner

@Astrrra Astrrra Awaiting requested review from Astrrra

@Skorpionm Skorpionm Awaiting requested review from Skorpionm Skorpionm is a code owner

Assignees

No one assigned

Labels

App Loader FAP loader + API-related Core+Services HAL, furi & core system services

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@skotopes @hedger @DrZlo13