[clang-tools-extra] [clang-doc] Add --asset option to clang-doc (PR #94717)

Paul Kirth via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 7 09:37:10 PDT 2024


================
@@ -131,12 +137,55 @@ std::string GetExecutablePath(const char *Argv0, void *MainAddr) {
   return llvm::sys::fs::getMainExecutable(Argv0, MainAddr);
 }
 
+void GetAssetFiles(clang::doc::ClangDocContext &CDCtx) {
+  std::error_code Code;
----------------
ilovepi wrote:

nit: I'd use `Err`. 

There isn't a completely consistent practice in Clang-Doc code, but `FileErr` seems most common. https://llvm.org/docs/ProgrammersManual.html#interoperability-with-std-error-code-and-erroror has a little to say, and the `Error handling` section on that page may be useful, but we don't have a firm convention.

https://github.com/llvm/llvm-project/pull/94717


More information about the cfe-commits mailing list