Skip to content
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

Additional functions for the Char module to check common character properties #467

Merged
merged 4 commits into from
Jan 4, 2025

Conversation

mobotsar
Copy link
Contributor

@mobotsar mobotsar commented Jan 3, 2025

I've added some functions to check whether a given character is uppercase, lowercase, a letter, a digit, or white-space, respectively. These are particularly useful in building predicates for CCString functions like contains, exists, &c. Let me know what you think, and if you want any changes.

The names align with the convention set by Stdlib's Char of indicating the character encoding nominally.

@mobotsar
Copy link
Contributor Author

mobotsar commented Jan 3, 2025

It looks like there's some issue in the CI with missing darcs.

@c-cube
Copy link
Owner

c-cube commented Jan 4, 2025

It looks good overall, good idea! Can you rebase instead of having a merge commit? I'd also like at least a few tests on this one. I did look at the ascii table, but tests don't hurt :)

@mobotsar
Copy link
Contributor Author

mobotsar commented Jan 4, 2025

Thanks. I rebased and will push tomorrow after adding some tests.

@mobotsar
Copy link
Contributor Author

mobotsar commented Jan 4, 2025

Sorry about the git mess; I had a rough morning... Should all be good now.

@c-cube c-cube merged commit 31ad563 into c-cube:main Jan 4, 2025
8 checks passed
@c-cube
Copy link
Owner

c-cube commented Jan 4, 2025

That's great, thanks!

Sorry about the git mess; I had a rough morning... Should all be good now.

yeah that happens 😁

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