feat: configurable extraction of inaccessible bases

cppalliance · Nov 27, 2023 · a77752e · a77752e
1 parent b8b6880
commit a77752e
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 4 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -344,6 +344,7 @@ jobs:
             "generate: %s"                     \
             "multipage: %s"                    \
             "inaccessible-members: never"      \
+            "inaccessible-bases: never"        \
             "filters:"                         \
             "  symbols:"                       \
             "    exclude:"                     \

diff --git a/docs/modules/ROOT/pages/config-file.adoc b/docs/modules/ROOT/pages/config-file.adoc
@@ -84,10 +84,10 @@ when set to `dependency`.
 |No
 
 |inaccessible-members
-|Whether to extract inaccessible members. Currently unimplemented.
+|Whether to extract private/protected members. `always` or `never`.
 |No
 
 |inaccessible-bases
-|Whether to extract inaccessible bases. Currently unimplemented.
+|Whether to extract private/protected bases. `always` or `never`.
 
 |===
diff --git a/src/lib/AST/ASTVisitor.cpp b/src/lib/AST/ASTVisitor.cpp
@@ -1945,14 +1945,25 @@ class ASTVisitor
         {
             for(const CXXBaseSpecifier& B : D->bases())
             {
+                AccessSpecifier access = B.getAccessSpecifier();
+                // KRYSTIAN FIXME: we need finer-grained control
+                // for protected bases, since an inheriting class
+                // will have access to the bases public members...
+                if(config_->inaccessibleBases !=
+                    ConfigImpl::SettingsImpl::ExtractPolicy::Always)
+                {
+                    if(access == AccessSpecifier::AS_private ||
+                        access == AccessSpecifier::AS_protected)
+                        continue;
+                }
+
                 I.Bases.emplace_back(
                     // the extraction of the base type is
                     // performed in direct dependency mode
                     buildTypeInfo(
                         B.getType(),
                         ExtractMode::DirectDependency),
-                    convertToAccessKind(
-                        B.getAccessSpecifier()),
+                    convertToAccessKind(access),
                     B.isVirtual());
             }
         }