[PATCH] D38583: [clangd] Added async API to run code completion.

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 5 07:57:45 PDT 2017


ilya-biryukov added inline comments.


================
Comment at: clangd/ClangdServer.h:236-237
+  ///
+  /// Request is processed  asynchronously, you can use returned future to wait
+  /// for results of async request.
+  ///
----------------
klimek wrote:
> Extra space before asynchronously.
> Replace first ',' with ';'
> ... wait for the results of the async request...
Split into two sentences on `,` instead of replacing it with `;`. Feels even better.


https://reviews.llvm.org/D38583





More information about the cfe-commits mailing list