diff options
| author | Mara Bos <m-ou.se@m-ou.se> | 2021-02-04 21:10:37 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-02-04 21:10:37 +0100 |
| commit | 21e5827800b9962a035791bb55e857df33a2f681 (patch) | |
| tree | 695008e6b179bc3d083f653e9e5aaa735d6e531e /compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp | |
| parent | 87b269ab6654ddcaed43933ebcd3754d000117b7 (diff) | |
| parent | 4d1efb751a8fd70a38c695a6047a837d115b8af5 (diff) | |
| download | rust-21e5827800b9962a035791bb55e857df33a2f681.tar.gz rust-21e5827800b9962a035791bb55e857df33a2f681.zip | |
Rollup merge of #81710 - TyPR124:patch-2, r=m-ou-se
OsStr eq_ignore_ascii_case takes arg by value
Per a comment on #70516 this changes `eq_ignore_ascii_case` to take the generic parameter `S: AsRef<OsStr>` by value instead of by reference.
This is technically a breaking change to an unstable method. I think the only way it would break is if you called this method with an explicit type parameter, ie `my_os_str.eq_ignore_ascii_case::<str>("foo")` becomes `my_os_str.eq_ignore_ascii_case::<&str>("foo")`.
Besides that, I believe it is overall more flexible since it can now take an owned `OsString` for example.
If this change should be made in some other PR (like #80193) then please just close this.
Diffstat (limited to 'compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions