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
erts: Support is_boolean/1 in match specs #7046
erts: Support is_boolean/1 in match specs #7046
Conversation
CT Test Results 3 files 133 suites 49m 0s ⏱️ Results for commit 1b787560. ♻️ This comment has been updated with latest results. To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass. See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally. Artifacts// Erlang/OTP Github Action Bot |
51e9d89
to
c6a25d7
Compare
Can you also add the BIFs to docs at https://www.erlang.org/doc/apps/erts/match_spec.html, |
…rlang/otp#7046. Still issues with is_record, but that's in our macro expansion land now.
…rlang/otp#7046. Still issues with is_record, but that's in our macro expansion land now.
Just finished testing all of these with |
c6a25d7
to
1b78756
Compare
Thanks, I've pushed an update with fixed documentation and extended tests. :-) |
This PR adds a few missing guard BIFs like
is_boolean/1
to the list of functions allowed in match specifications, and adds a test to ensure that all new guard BIFs are added to this list.Fixes #7045