diff options
| author | Nicholas Nethercote <n.nethercote@gmail.com> | 2024-01-09 09:08:49 +1100 |
|---|---|---|
| committer | Nicholas Nethercote <n.nethercote@gmail.com> | 2024-01-10 07:40:00 +1100 |
| commit | beeaee97859f34c7a618565e6ff8539b52ac3912 (patch) | |
| tree | e8e46d3c08b4270377bf84509224db53f5ec1452 /compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp | |
| parent | 89f511e4dd4cde330c85f7b596d3f68d381a582e (diff) | |
| download | rust-beeaee97859f34c7a618565e6ff8539b52ac3912.tar.gz rust-beeaee97859f34c7a618565e6ff8539b52ac3912.zip | |
Rename consuming chaining methods on `DiagnosticBuilder`.
In #119606 I added them and used a `_mv` suffix, but that wasn't great.
A `with_` prefix has three different existing uses.
- Constructors, e.g. `Vec::with_capacity`.
- Wrappers that provide an environment to execute some code, e.g.
`with_session_globals`.
- Consuming chaining methods, e.g. `Span::with_{lo,hi,ctxt}`.
The third case is exactly what we want, so this commit changes
`DiagnosticBuilder::foo_mv` to `DiagnosticBuilder::with_foo`.
Thanks to @compiler-errors for the suggestion.
Diffstat (limited to 'compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions