ilya-biryukov added inline comments.
================ Comment at: clangd/TUScheduler.h:77 +class DiagnosticsResult { +public: ---------------- To avoid boilerplate this could be changed to: ``` using DiagnosticsResult = Optional<vector<Diag>>; ``` Would still keep it a named typedef to allow documenting what the `None` value means. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D53946 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits