-
-
Notifications
You must be signed in to change notification settings - Fork 768
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
ICU-22879 Provide collator based predicates for use with STL algorithms #3149
Conversation
82c9a62
to
e019452
Compare
Notice: the branch changed across the force-push!
~ Your Friendly Jira-GitHub PR Checker Bot |
assertTrue("string", equal_to(std::string{TEXT_CHAR}, TEXT_CHAR)); | ||
} | ||
|
||
void CollationTest::TestCollatorMap() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to see how things would work with std::map
, so I tried it out.
Might be worth putting examples using that to the API documentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Examples would be very good but not a blocker for this PR. Happy to re-approve with more examples, and also happy to approve a follow-up.
assertTrue("string", equal_to(std::string{TEXT_CHAR}, TEXT_CHAR)); | ||
} | ||
|
||
void CollationTest::TestCollatorMap() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Examples would be very good but not a blocker for this PR. Happy to re-approve with more examples, and also happy to approve a follow-up.
Notice: the branch changed across the force-push!
~ Your Friendly Jira-GitHub PR Checker Bot |
Hooray! The files in the branch are the same across the force-push. 😃 ~ Your Friendly Jira-GitHub PR Checker Bot |
Checklist
ALLOW_MANY_COMMITS=true