Page MenuHomePhabricator

[clangd] Simplify code of ClangdLSPServer::onCommand
ClosedPublic

Authored by ilya-biryukov on Aug 16 2019, 5:23 AM.

Details

Summary

By inlining a complicated lambda into its single call-site.

Also ensure we call Reply() exactly once even if tweaks return both
ShowMessage and ApplyEdit effects.

Diff Detail

Repository
rL LLVM

Event Timeline

ilya-biryukov created this revision.Aug 16 2019, 5:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 16 2019, 5:23 AM
hokein accepted this revision.Aug 16 2019, 5:39 AM

thanks.

This revision is now accepted and ready to land.Aug 16 2019, 5:39 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptAug 16 2019, 5:49 AM