Skip to content
This repository was archived by the owner on Dec 29, 2022. It is now read-only.

Conversation

@ehuss
Copy link
Contributor

@ehuss ehuss commented Jul 14, 2018

Fixes #838

@alexheretic
Copy link
Contributor

Thanks! I can confirm this does fix #838

@Xanewok
Copy link
Contributor

Xanewok commented Jul 14, 2018

Nice catch! Since we're using the rls as rustc shim, it's only fair to use the same parameters as rustc binary itself.

link to rustc.rs rationale

@Xanewok Xanewok merged commit f07b789 into rust-lang:master Jul 14, 2018
@nrc
Copy link
Member

nrc commented Jul 17, 2018

@ehuss Thanks so much for this PR! I had been dreading trying to fix this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants