-
Notifications
You must be signed in to change notification settings - Fork 57
Shishkina Valeriya. Lab #1. Var #1. #18
Shishkina Valeriya. Lab #1. Var #1. #18
Conversation
#include "clang/Frontend/CompilerInstance.h" | ||
#include "clang/Frontend/FrontendPluginRegistry.h" | ||
|
||
class PrintClassesVisitor : public clang::RecursiveASTVisitor<PrintClassesVisitor> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix diff suggestions of clang-format test: https://github.com/NN-complr-tech/llvm-nnsu-2024/actions/runs/8310037939/job/22742071220?pr=18#step:4:14
|
||
protected: | ||
bool ParseArgs(const clang::CompilerInstance &Compiler, | ||
const std::vector<std::string> &args) override { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please fix clang-tidy warnings if applicable: https://github.com/NN-complr-tech/llvm-nnsu-2024/actions/runs/8310037939/job/22742071140?pr=18#step:6:489
}; | ||
|
||
// CHECK: Test4 | ||
struct Test4{}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a test for help
argument.
@ShLera04 Arseniy have already approved the PR. But anyway could you please fix the test? |
As a result of the laboratory work, a plugin was written that outputs the names of all classes and their fields.
For example:
Input: