@@ -316,7 +316,7 @@ internal Binding Bind(IntPtr inst, IntPtr args, IntPtr kw, MethodBase info, Meth
316
316
Runtime . XDecref ( valueList ) ;
317
317
}
318
318
319
- var pynargs = ( int ) Runtime . PyTuple_Size ( args ) ;
319
+ var numPyArgs = ( int ) Runtime . PyTuple_Size ( args ) ;
320
320
var isGeneric = false ;
321
321
if ( info != null )
322
322
{
@@ -343,14 +343,14 @@ internal Binding Bind(IntPtr inst, IntPtr args, IntPtr kw, MethodBase info, Meth
343
343
int kwargsMatched ;
344
344
int defaultsNeeded ;
345
345
bool isOperator = OperatorMethod . IsOperatorMethod ( mi ) ; // e.g. op_Addition is defined for OperableObject
346
- if ( ! MatchesArgumentCount ( pynargs , pi , kwargDict , isOperator , out paramsArray , out defaultArgList , out kwargsMatched , out defaultsNeeded ) )
346
+ if ( ! MatchesArgumentCount ( numPyArgs , pi , kwargDict , isOperator , out paramsArray , out defaultArgList , out kwargsMatched , out defaultsNeeded ) )
347
347
{
348
348
continue ;
349
349
}
350
350
var outs = 0 ;
351
- int clrnargs = pi . Length ;
352
- isOperator = isOperator && pynargs == clrnargs - 1 ; // Handle mismatched arg numbers due to Python operator being bound.
353
- var margs = TryConvertArguments ( pi , paramsArray , args , pynargs , kwargDict , defaultArgList ,
351
+ int numClrArgs = pi . Length ;
352
+ isOperator = isOperator && numPyArgs == numClrArgs - 1 ; // Handle mismatched arg numbers due to Python operator being bound.
353
+ var margs = TryConvertArguments ( pi , paramsArray , args , numPyArgs , kwargDict , defaultArgList ,
354
354
needsResolution : _methods . Length > 1 , // If there's more than one possible match.
355
355
isOperator : isOperator ,
356
356
outs : out outs ) ;
@@ -669,7 +669,7 @@ static Type TryComputeClrArgumentType(Type parameterType, IntPtr argument, bool
669
669
/// <summary>
670
670
/// Check whether the number of Python and .NET arguments match, and compute additional arg information.
671
671
/// </summary>
672
- /// <param name="pynargs ">Number of positional args passed from Python.</param>
672
+ /// <param name="numPyArgs ">Number of positional args passed from Python.</param>
673
673
/// <param name="parameters">Parameters of the specified .NET method.</param>
674
674
/// <param name="kwargDict">Keyword args passed from Python.</param>
675
675
/// <param name="isOperator">True if the parameters' method is an operator.</param>
@@ -678,7 +678,7 @@ static Type TryComputeClrArgumentType(Type parameterType, IntPtr argument, bool
678
678
/// <param name="kwargsMatched">Number of kwargs from Python that are also present in the .NET method.</param>
679
679
/// <param name="defaultsNeeded">Number of non-null defaultsArgs.</param>
680
680
/// <returns></returns>
681
- static bool MatchesArgumentCount ( int pynargs , ParameterInfo [ ] parameters ,
681
+ static bool MatchesArgumentCount ( int numPyArgs , ParameterInfo [ ] parameters ,
682
682
Dictionary < string , IntPtr > kwargDict ,
683
683
bool isOperator ,
684
684
out bool paramsArray ,
@@ -688,15 +688,15 @@ static bool MatchesArgumentCount(int pynargs, ParameterInfo[] parameters,
688
688
{
689
689
defaultArgList = null ;
690
690
var match = false ;
691
- int clrnargs = parameters . Length ;
692
- paramsArray = clrnargs > 0 ? Attribute . IsDefined ( parameters [ clrnargs - 1 ] , typeof ( ParamArrayAttribute ) ) : false ;
691
+ int numClrArgs = parameters . Length ;
692
+ paramsArray = numClrArgs > 0 ? Attribute . IsDefined ( parameters [ numClrArgs - 1 ] , typeof ( ParamArrayAttribute ) ) : false ;
693
693
kwargsMatched = 0 ;
694
694
defaultsNeeded = 0 ;
695
- if ( pynargs == clrnargs && kwargDict . Count == 0 )
695
+ if ( numPyArgs == numClrArgs && kwargDict . Count == 0 )
696
696
{
697
697
match = true ;
698
698
}
699
- else if ( pynargs < clrnargs && ( ! paramsArray || pynargs == clrnargs - 1 ) )
699
+ else if ( numPyArgs < numClrArgs && ( ! paramsArray || numPyArgs == numClrArgs - 1 ) )
700
700
{
701
701
match = true ;
702
702
// operator methods will have 2 CLR args but only one Python arg,
@@ -711,7 +711,7 @@ static bool MatchesArgumentCount(int pynargs, ParameterInfo[] parameters,
711
711
// a corresponding keyword arg or a default param, unless the method
712
712
// method accepts a params array (which cannot have a default value)
713
713
defaultArgList = new ArrayList ( ) ;
714
- for ( var v = pynargs ; v < clrnargs ; v ++ )
714
+ for ( var v = numPyArgs ; v < numClrArgs ; v ++ )
715
715
{
716
716
if ( kwargDict . ContainsKey ( parameters [ v ] . Name ) )
717
717
{
@@ -736,8 +736,8 @@ static bool MatchesArgumentCount(int pynargs, ParameterInfo[] parameters,
736
736
}
737
737
}
738
738
}
739
- else if ( pynargs > clrnargs && clrnargs > 0 &&
740
- Attribute . IsDefined ( parameters [ clrnargs - 1 ] , typeof ( ParamArrayAttribute ) ) )
739
+ else if ( numPyArgs > numClrArgs && numClrArgs > 0 &&
740
+ Attribute . IsDefined ( parameters [ numClrArgs - 1 ] , typeof ( ParamArrayAttribute ) ) )
741
741
{
742
742
// This is a `foo(params object[] bar)` style method
743
743
match = true ;