Skip to content

internal: Add delete_whitespace() to SyntaxEditor - #22949

Open
A4-Tacks wants to merge 4 commits into
rust-lang:masterfrom
A4-Tacks:del-ws-api
Open

internal: Add delete_whitespace() to SyntaxEditor#22949
A4-Tacks wants to merge 4 commits into
rust-lang:masterfrom
A4-Tacks:del-ws-api

Conversation

@A4-Tacks

Copy link
Copy Markdown
Member
  • And Fix completions trait_impl desugar whitespace

Example

Before this PR

impl DesugaredAsyncTrait for () {
      fn foo(&self) -> impl Future<Output = usize> {
         $0
     }
}

After this PR

impl DesugaredAsyncTrait for () {
    fn foo(&self) -> impl Future<Output = usize> {
        $0
    }
}

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 29, 2026
@rustbot

This comment has been minimized.

Example
---
**Before this PR**

```rust
impl DesugaredAsyncTrait for () {
      fn foo(&self) -> impl Future<Output = usize> {
         $0
     }
}
```

**After this PR**

```rust
impl DesugaredAsyncTrait for () {
    fn foo(&self) -> impl Future<Output = usize> {
        $0
    }
}
```
@rustbot

rustbot commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

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

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants