[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