Merge pull request #237 from andreasfertig/fixIssue236

Fixed #236: Add missing virtual specifier for virtual inheritance.

CodeGenerator.cpp

@@ -2370,7 +2370,10 @@ void CodeGenerator::InsertArg(const CXXRecordDecl* stmt)
         mOutputFormatHelper.Append(" : ");
 
         ForEachArg(stmt->bases(), [&](const auto& base) {
-            mOutputFormatHelper.Append(AccessToString(base.getAccessSpecifier()), " ", GetName(base.getType()));
+            const std::string virtualKw{base.isVirtual() ? kwVirtualSpace : ""};
+
+            mOutputFormatHelper.Append(
+                AccessToString(base.getAccessSpecifier()), " ", virtualKw, GetName(base.getType()));
         });
     }
 

tests/Issue236.cpp

@@ -0,0 +1,2 @@
+struct S {};
+class T : virtual S {};

tests/Issue236.expect

@@ -0,0 +1,17 @@
+struct S
+{
+  // inline constexpr S & operator=(const S &) = default;
+  // inline constexpr S & operator=(S &&) = default;
+  // inline ~S() = default;
+};
+
+
+class T : private virtual S
+{
+  public: 
+  // inline T & operator=(const T &) = default;
+  // inline T & operator=(T &&) = default;
+  // inline ~T() = default;
+};
+
+