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

Reply via email to