Skip to content

Comments about btstack_run_loop_execute #388

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

Merged

Conversation

peterharperuk
Copy link
Contributor

Add some comments about how btstack_run_loop_execute does nothing in threadsafe backrgound mode.

Add some comments about how btstack_run_loop_execute does nothing in
threadsafe backrgound mode.
@peterharperuk peterharperuk requested a review from kilograham May 26, 2023 15:29
// We're using threadsafe background where poll does nothing and work is performed in a low priority IRQ
// So btstack_run_loop_execute effectively does nothing and you're free to just loop and do your own stuff
// You would have to call btstack_run_loop_execute if you were using pico_cyw43_arch_poll
#if 1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies for the stupid question, but why does this PR add #if 1 in client.c but #if 0 in server.c ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To demonstrate that both methods work?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough 🙂

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, i also found it a bit odd(confusing); i'm planning to make some comment tweaks when i merge

@peterharperuk
Copy link
Contributor Author

The changes look ok to me

@kilograham kilograham merged commit 5f28220 into raspberrypi:develop Jun 6, 2023
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.

3 participants