chore: separate ASTVisitor from SemaConsumer

Author: sdkrystian

include/mrdox/Metadata/Function.hpp

@@ -137,8 +137,6 @@ struct FunctionInfo
     : IsInfo<InfoKind::Function>
     , SourceInfo
 {
-    friend class ASTVisitor;
-
     std::unique_ptr<TypeInfo> ReturnType; // Info about the return type of this function.
     std::vector<Param> Params; // List of parameters.
 
@@ -159,14 +157,6 @@ struct FunctionInfo
         : IsInfo(ID)
     {
     }
-
-private:
-    explicit
-    FunctionInfo(
-        std::unique_ptr<TemplateInfo>&& T)
-        : Template(std::move(T))
-    {
-    }
 };
 
 //------------------------------------------------

include/mrdox/Metadata/Record.hpp

@@ -77,8 +77,6 @@ struct RecordInfo
     : IsInfo<InfoKind::Record>
     , SourceInfo
 {
-    friend class ASTVisitor;
-
     /** Kind of record this is (class, struct, or union).
     */
     RecordKeyKind KeyKind = RecordKeyKind::Struct;
@@ -119,14 +117,6 @@ struct RecordInfo
         : IsInfo(ID)
     {
     }
-
-private:
-    explicit
-    RecordInfo(
-        std::unique_ptr<TemplateInfo>&& T)
-        : Template(std::move(T))
-    {
-    }
 };
 
 } // mrdox

include/mrdox/Metadata/Typedef.hpp

@@ -27,8 +27,6 @@ struct TypedefInfo
     : IsInfo<InfoKind::Typedef>
     , SourceInfo
 {
-    friend class ASTVisitor;
-
     std::unique_ptr<TypeInfo> Type;
 
     // Indicates if this is a new C++ "using"-style typedef:
@@ -47,16 +45,6 @@ struct TypedefInfo
         : IsInfo(ID)
     {
     }
-
-private:
-    // KRYSTIAN NOTE: if Template is non-null,
-    // then IsUsing *must* be set; should we do it here?
-    explicit
-    TypedefInfo(
-        std::unique_ptr<TemplateInfo>&& T)
-        : Template(std::move(T))
-    {
-    }
 };
 
 } // mrdox

include/mrdox/Metadata/Variable.hpp

@@ -41,8 +41,6 @@ struct VariableInfo
     : IsInfo<InfoKind::Variable>
     , SourceInfo
 {
-    friend class ASTVisitor;
-
     /** The type of the variable */
     std::unique_ptr<TypeInfo> Type;
 
@@ -58,15 +56,6 @@ struct VariableInfo
         : IsInfo(ID)
     {
     }
-
-private:
-    explicit
-    VariableInfo(
-        std::unique_ptr<TemplateInfo>&& T) noexcept
-        : Template(std::move(T))
-    {
-
-    }
 };
 
 } // mrdox