[clang-tools-extra] [clang-doc] add more test to clang-doc (PR #97518)

2024-07-03 Thread via cfe-commits
https://github.com/PeterChou1 edited https://github.com/llvm/llvm-project/pull/97518 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] add more test to clang-doc (PR #97518)

2024-07-02 Thread via cfe-commits
@@ -0,0 +1,701 @@ +// RUN: sed 's|$test_dir|%/S|g' %S/Inputs/advance-project/database_template.json > %t/build/compile_commands.json +// RUN: clang-doc --format=html --doxygen --output=%t/docs --executor=all-TUs %t/build/compile_commands.json +// RUN: clang-doc --format=md --do

[clang-tools-extra] [clang-doc] add more test to clang-doc (PR #97518)

2024-07-02 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,31 @@ +#include "Array.h" + +// Implementation of Array + +/** +* Initializes all elements of the array to their default value. +*/ +template +Array::Array() { + // Implementation stub +} + +/** +* Array access operator for Array +* Provides read and write access to

[clang-tools-extra] [clang-doc] add more test to clang-doc (PR #97518)

2024-07-02 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi commented: Thanks for adding more tests. Clang-Doc badly needs more comprehensive test coverage. I have a few concerns, however, about the testing strategy. Largely, these tests seem to check the exact output, rather than spot checking for particular features. What

[clang-tools-extra] [clang-doc] add more test to clang-doc (PR #97518)

2024-07-02 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,701 @@ +// RUN: sed 's|$test_dir|%/S|g' %S/Inputs/advance-project/database_template.json > %t/build/compile_commands.json +// RUN: clang-doc --format=html --doxygen --output=%t/docs --executor=all-TUs %t/build/compile_commands.json +// RUN: clang-doc --format=md --do

[clang-tools-extra] [clang-doc] add more test to clang-doc (PR #97518)

2024-07-02 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,701 @@ +// RUN: sed 's|$test_dir|%/S|g' %S/Inputs/advance-project/database_template.json > %t/build/compile_commands.json +// RUN: clang-doc --format=html --doxygen --output=%t/docs --executor=all-TUs %t/build/compile_commands.json +// RUN: clang-doc --format=md --do

[clang-tools-extra] [clang-doc] add more test to clang-doc (PR #97518)

2024-07-02 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi edited https://github.com/llvm/llvm-project/pull/97518 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] add more test to clang-doc (PR #97518)

2024-07-02 Thread via cfe-commits
https://github.com/PeterChou1 edited https://github.com/llvm/llvm-project/pull/97518 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] add more test to clang-doc (PR #97518)

2024-07-02 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tools-extra Author: None (PeterChou1) Changes This patches adds more e2e test to clang-doc which tests the html and markdown output. I've made the following changes - Modified basic-project.test to also include the markdown output - Added namespa