Page MenuHomePhabricator

[clangd] Return TextEdits from ClangdServer::applyTweak

Authored by ilya-biryukov on Tue, Jun 11, 7:31 AM.



Instead of tooling::Replacements. So that embedders do not need to store
the contents of the file.

This also aligns better with ClangdServer::rename.

Diff Detail


Event Timeline

ilya-biryukov created this revision.Tue, Jun 11, 7:31 AM
Herald added a project: Restricted Project. · View Herald TranscriptTue, Jun 11, 7:31 AM
ilya-biryukov edited the summary of this revision. (Show Details)Tue, Jun 11, 7:35 AM

@hokein just realized you might be a better reviewer, since this makes applyTweak aligned with rename. And you implemented rename in the first place

hokein accepted this revision.Wed, Jun 12, 4:49 AM

looks good.

This revision is now accepted and ready to land.Wed, Jun 12, 4:49 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptWed, Jun 12, 5:00 AM