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

Use lines_any() when parsing output form "ar" #12475

Closed
wants to merge 1 commit into from

Conversation

alexcrichton
Copy link
Member

On windows lines are delimited with \r\n while on unix they're delimited with
\n.

cc #12471

On windows lines are delimited with \r\n while on unix they're delimited with
\n.

cc rust-lang#12471
@brson
Copy link
Contributor

brson commented Feb 22, 2014

I wonder why lines() doesn't just deal with this. This seems like a bug that every cross-platform app that splits strings into lines will hit.

@brson
Copy link
Contributor

brson commented Feb 22, 2014

*Any cross-platform app that reads text files. Maybe this isn't that pervasive a problem.

@bors bors closed this in 4243cad Feb 25, 2014
@alexcrichton alexcrichton deleted the issue-12471 branch February 27, 2014 20:13
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 25, 2022
fix: Fix trait impl completions using wrong insert position

Fixes rust-lang/rust-analyzer#12474
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.

3 participants