[PATCH] D39430: [clangd] various fixes
    Marc-Andre Laperle via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Sun Dec  3 21:13:11 PST 2017
    
    
  
malaperle added a comment.
> - Look up in parent directories for a .clang-format file. Use "LLVM" as fallback style.
> - Move cached fixits to ClangdServer, remove fixits once the client closes the document.
> - Fix translations between LSP rows/cols (0-based) and clang row/cols (1-based).
I'd rather keep one commit per fix or feature. I would think the clang-format part should be one commit/review on its own. But I'm not very familiar with the practices here so I'll let others comment.
Repository:
  rCTE Clang Tools Extra
https://reviews.llvm.org/D39430
    
    
More information about the cfe-commits
mailing list