Skip to content

async support with feature - #30

Open
tommasoclini wants to merge 2 commits into
funbiscuit:mainfrom
tommasoclini:maybe_async
Open

async support with feature#30
tommasoclini wants to merge 2 commits into
funbiscuit:mainfrom
tommasoclini:maybe_async

Conversation

@tommasoclini

@tommasoclini tommasoclini commented Jul 30, 2026

Copy link
Copy Markdown

add async processor support using maybe_async_cfg2 on feature "async".

I tried running memory.sh and I'm getting weird results, there are 200 more bytes in every measure in every test, but when comparing actual binary size with cargo size or avr-size, the main branch and this branch are equal.
I used this command(avr-objcopy -O ihex binary.elf binary.hex) to get the raw binary, and the two files(old and new) do not differ.

@tommasoclini

Copy link
Copy Markdown
Author

@Finomnis what do you think?

@Finomnis

Finomnis commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Features should enable capabilities but they should not remove them (like what you do with the original sync function)

Reason is that other libraries might enable features out of your control.

But that's a general guideline, if you say this crate is enduser specific, then maybe this might be a solution, yes.

@tommasoclini tommasoclini mentioned this pull request Jul 31, 2026
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.

2 participants