Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: refactor whitespace check in createWorklet function #146

Merged
merged 1 commit into from
Mar 12, 2024

Conversation

jdnichollsc
Copy link
Contributor

Extracted the lambda function used in the std::all_of call into a separate named function.

Closes #144

Extracted the lambda function used in the std::all_of call into a separate named function.
@spsaucier
Copy link

Fixes the issue for me.

@jdnichollsc
Copy link
Contributor Author

let me know what you think folks, I'm just learning this package! 🫡

@adonaiKasawa
Copy link

adonaiKasawa commented Mar 11, 2024

I resolved the error with the following code:
you must first include the following headers

Capture d’écran 2024-03-11 à 16 07 42

then look for the line where we find
bool isCodeEmpty = std::all_of(_code.begin(), _code.end(), std::isspace);

replace you with
bool isCodeEmpty = std::all_of(_code.begin(), _code.end(), [](unsigned char c) { return std::isspace(c); });

Copy link
Member

@mrousavy mrousavy left a comment

Choose a reason for hiding this comment

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

looks good, nice work!

@mrousavy mrousavy merged commit d3d54ba into margelo:main Mar 12, 2024
5 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

error: no matching function for call to 'all_of'
5 participants