Corrected the access modifier of overridden methods because in C++ overriding may change access.

Signed-off-by: Dimitar Dobrev <dpldobrev@yahoo.com>

src/AST/Class.cs

@@ -201,6 +201,22 @@ namespace CppSharp.AST
             }
         }
 
+        public Method GetRootBaseMethod(Method @override)
+        {
+            return (from @base in Bases
+                    let baseMethod = (
+                        from method in @base.Class.Methods
+                        where
+                            method.Name == @override.Name &&
+                            method.ReturnType == @override.ReturnType &&
+                            method.Parameters.Count == @override.Parameters.Count &&
+                            method.Parameters.SequenceEqual(@override.Parameters,
+                                                            new ParameterTypeComparer())
+                        select method).FirstOrDefault()
+                    let rootBaseMethod = @base.Class.GetRootBaseMethod(@override)
+                    select rootBaseMethod ?? baseMethod).FirstOrDefault();
+        }
+
         public override T Visit<T>(IDeclVisitor<T> visitor)
         {
             return visitor.VisitClassDecl(this);

src/AST/Function.cs

@@ -60,6 +60,19 @@ namespace CppSharp.AST
         }
     }
 
+    public class ParameterTypeComparer : IEqualityComparer<Parameter>
+    {
+        public bool Equals(Parameter x, Parameter y)
+        {
+            return x.QualifiedType == y.QualifiedType;
+        }
+
+        public int GetHashCode(Parameter obj)
+        {
+            return obj.Type.GetHashCode();
+        }
+    }
+
     public enum FunctionSynthKind
     {
         None,

src/Generator/Generators/CSharp/CSharpTextTemplate.cs

@@ -1483,7 +1483,7 @@ namespace CppSharp.Generators.CSharp
             PushBlock(CSharpBlockKind.Method);
             GenerateDeclarationCommon(method);
 
-            switch (method.Access)
+            switch (GetValidMethodAccess(method, @class))
             {
                 case AccessSpecifier.Public:
                     Write(Driver.Options.GenerateAbstractImpls && @class.IsAbstract
@@ -1578,6 +1578,20 @@ namespace CppSharp.Generators.CSharp
             PopBlock(NewLineKind.BeforeNextBlock);
         }
 
+        private static AccessSpecifier GetValidMethodAccess(Method method, Class @class)
+        {
+            switch (method.Access)
+            {
+                case AccessSpecifier.Public:
+                    return @class.IsAbstract && method.IsConstructor ?
+                        AccessSpecifier.Protected : AccessSpecifier.Public;
+                case AccessSpecifier.Protected:
+                    return method.IsOverride ?
+                        @class.GetRootBaseMethod(method).Access : AccessSpecifier.Protected;
+            }
+            return AccessSpecifier.Private;
+        }
+
         private void GenerateVirtualTableFunctionCall(Function method, Class @class)
         {
             string delegateId;

src/Generator/Utils/ParameterTypeComparer.cs

@@ -1,18 +0,0 @@
-using System.Collections.Generic;
-using CppSharp.AST;
-
-namespace CppSharp.Utils
-{
-    public class ParameterTypeComparer : IEqualityComparer<Parameter>
-    {
-        public bool Equals(Parameter x, Parameter y)
-        {
-            return x.QualifiedType == y.QualifiedType;
-        }
-
-        public int GetHashCode(Parameter obj)
-        {
-            return obj.Type.GetHashCode();
-        }
-    }
-}