Address peer review

Author: jhonabreul

src/runtime/ClassManager.cs

@@ -454,17 +454,17 @@ private static ClassInfo GetClassInfo(Type type, ClassBase impl)
                         }
                         methodList.Add(meth);
 
-                        if (!meth.IsSpecialName && !OperatorMethod.IsOperatorMethod(meth))
+                        if (!OperatorMethod.IsOperatorMethod(meth))
                         {
                             var snakeCasedName = name.ToSnakeCase();
                             if (snakeCasedName != name)
                             {
                                 if (!methods.TryGetValue(snakeCasedName, out methodList))
-                            {
+                                {
                                     methodList = methods[snakeCasedName] = new MethodOverloads(false);
+                                }
+                                methodList.Add(meth);
                             }
-                            methodList.Add(meth);
-                        }
                         }
                         continue;
 
@@ -506,8 +506,9 @@ private static ClassInfo GetClassInfo(Type type, ClassBase impl)
                         }
 
                         ob = new PropertyObject(pi);
+                        var allocatedOb = ob.AllocObject();
                         ci.members[pi.Name] = ob.AllocObject();
-                        ci.members[pi.Name.ToSnakeCase()] = ob.AllocObject();
+                        ci.members[pi.Name.ToSnakeCase()] = allocatedOb;
                         continue;
 
                     case MemberTypes.Field:
@@ -517,14 +518,15 @@ private static ClassInfo GetClassInfo(Type type, ClassBase impl)
                             continue;
                         }
                         ob = new FieldObject(fi);
-                        ci.members[mi.Name] = ob.AllocObject();
+                        allocatedOb = ob.AllocObject();
+                        ci.members[mi.Name] = allocatedOb;
 
                         var pepName = fi.Name.ToSnakeCase();
                         if (fi.IsLiteral)
                         {
                             pepName = pepName.ToUpper();
                         }
-                        ci.members[pepName] = ob.AllocObject();
+                        ci.members[pepName] = allocatedOb;
                         continue;
 
                     case MemberTypes.Event:
@@ -536,8 +538,9 @@ private static ClassInfo GetClassInfo(Type type, ClassBase impl)
                         ob = ei.AddMethod.IsStatic
                             ? new EventBinding(ei)
                             : new EventObject(ei);
-                        ci.members[ei.Name] = ob.AllocObject();
-                        ci.members[ei.Name.ToSnakeCase()] = ob.AllocObject();
+                        allocatedOb = ob.AllocObject();
+                        ci.members[ei.Name] = allocatedOb;
+                        ci.members[ei.Name.ToSnakeCase()] = allocatedOb;
                         continue;
 
                     case MemberTypes.NestedType:

src/runtime/MethodBinder.cs

@@ -441,6 +441,7 @@ internal Binding Bind(BorrowedReference inst, BorrowedReference args, BorrowedRe
                     kwArgDict[keyStr!] = new PyObject(value);
                 }
             }
+            var hasNamedArgs = kwArgDict != null && kwArgDict.Count > 0;
 
             // Fetch our methods we are going to attempt to match and bind too.
             var methods = info == null ? GetMethods()
@@ -452,7 +453,8 @@ internal Binding Bind(BorrowedReference inst, BorrowedReference args, BorrowedRe
                 // Relevant method variables
                 var mi = methodInformation.MethodBase;
                 var pi = methodInformation.ParameterInfo;
-                var paramNames = methodInformation.ParametersNames;
+                // Avoid accessing the parameter names property unless necessary
+                var paramNames = hasNamedArgs ? methodInformation.ParameterNames : Array.Empty<string>();
                 int pyArgCount = (int)Runtime.PyTuple_Size(args);
 
                 // Special case for operators
@@ -993,7 +995,7 @@ internal class MethodInformation
 
             public bool IsOriginal { get; }
 
-            public string[] ParametersNames { get { return _parametersNames.Value; } }
+            public string[] ParameterNames { get { return _parametersNames.Value; } }
 
             public MethodInformation(MethodBase methodBase, ParameterInfo[] parameterInfo)
                 : this(methodBase, parameterInfo, true)

src/runtime/Types/OperatorMethod.cs

@@ -27,6 +27,7 @@ public SlotDefinition(string methodName, int typeOffset)
             public int TypeOffset { get; }
 
         }
+        private static HashSet<string> _operatorNames;
         private static PyObject? _opType;
 
         static OperatorMethod()
@@ -63,6 +64,7 @@ static OperatorMethod()
                 ["op_LessThan"] = "__lt__",
                 ["op_GreaterThan"] = "__gt__",
             };
+            _operatorNames = new HashSet<string>(OpMethodMap.Keys.Concat(ComparisonOpMap.Keys));
         }
 
         public static void Initialize()
@@ -85,7 +87,7 @@ public static bool IsOperatorMethod(MethodBase method)
             {
                 return false;
             }
-            return OpMethodMap.ContainsKey(method.Name) || ComparisonOpMap.ContainsKey(method.Name);
+            return _operatorNames.Contains(method.Name);
         }
 
         public static bool IsComparisonOp(MethodBase method)