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

Fix build on latest Rust #63

Closed
wants to merge 1 commit into from
Closed

Conversation

crazymykl
Copy link
Contributor

to_uppercase returns an enumerator now. Adjust to this new reality.

@crazymykl crazymykl force-pushed the patch-1 branch 2 times, most recently from 61d82c9 to ea71d26 Compare March 13, 2015 03:06
`to_uppercase` returns an enumerator now. Adjust to this new reality.
used += m_char_upper.len_utf8();
let input_char_result = input_iter.next();
if input_char_result.is_none() ||
input_char_result.unwrap().to_uppercase() != m_char_upper {
input_char_result.unwrap().to_uppercase().next().unwrap() !=
m_char_upper {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I made this line into two because it made bootstrap.sh happy.

@kevinmehall
Copy link
Owner

Closing in favor of 31f6c99, which uses flat_map as recommended by rust-lang/rust#23126 to handle cases where a character uppercases to multiple characters. (This is why it was changed to an iterator). Thanks for your attempt, though!

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