Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commita376838

Browse files
committed
Use reverse instead of forward (semantics)
1 parente7da0bc commita376838

File tree

2 files changed

+13
-13
lines changed

2 files changed

+13
-13
lines changed

‎src/runtime/methodbinder.cs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -351,13 +351,13 @@ internal Binding Bind(IntPtr inst, IntPtr args, IntPtr kw, MethodBase info, Meth
351351
intclrnargs=pi.Length;
352352
isOperator=isOperator&&pynargs==clrnargs-1;// Handle mismatched arg numbers due to Python operator being bound.
353353
// Preprocessing pi to remove either the first or second argument.
354-
boolisForward=isOperator&&OperatorMethod.IsForward((MethodInfo)mi);// Only cast if isOperator.
355-
if(isOperator&&isForward){
354+
boolisReverse=isOperator&&OperatorMethod.IsReverse((MethodInfo)mi);// Only cast if isOperator.
355+
if(isOperator&&!isReverse){
356356
// The first Python arg is the right operand, while the bound instance is the left.
357357
// We need to skip the first (left operand) CLR argument.
358358
pi=pi.Skip(1).Take(1).ToArray();
359359
}
360-
elseif(isOperator&&!isForward){
360+
elseif(isOperator&&isReverse){
361361
// The first Python arg is the left operand.
362362
// We need to take the first CLR argument.
363363
pi=pi.Take(1).ToArray();
@@ -377,10 +377,10 @@ internal Binding Bind(IntPtr inst, IntPtr args, IntPtr kw, MethodBase info, Meth
377377
{
378378
// Postprocessing to extend margs.
379379
varmargsTemp=newobject[2];
380-
// Ifforward, thebound instance is the left operand.
381-
intboundOperandIndex=isForward?0:1;
382-
// Ifforward, thepassed instance is the right operand.
383-
intpassedOperandIndex=isForward?1:0;
380+
// Ifreverse, thepassed instance is the left operand.
381+
intpassedOperandIndex=isReverse?0:1;
382+
// Ifreverse, thebound instance is the right operand.
383+
intboundOperandIndex=isReverse?1:0;
384384
margsTemp[boundOperandIndex]=co.inst;
385385
margsTemp[passedOperandIndex]=margs[0];
386386
margs=margsTemp;

‎src/runtime/operatormethod.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -137,15 +137,15 @@ public static string ReversePyMethodName(string pyName)
137137
}
138138

139139
/// <summary>
140-
/// Check if the method is performing aforward orreverse operation.
140+
/// Check if the method is performing a reverse operation.
141141
/// </summary>
142142
/// <param name="method">The operator method.</param>
143143
/// <returns></returns>
144-
publicstaticboolIsForward(MethodInfomethod)
144+
publicstaticboolIsReverse(MethodInfomethod)
145145
{
146146
TypedeclaringType=method.DeclaringType;
147147
TypeleftOperandType=method.GetParameters()[0].ParameterType;
148-
returnleftOperandType==declaringType;
148+
returnleftOperandType!=declaringType;
149149
}
150150

151151
publicstaticvoidFilterMethods(MethodInfo[]methods,outMethodInfo[]forwardMethods,outMethodInfo[]reverseMethods)
@@ -154,12 +154,12 @@ public static void FilterMethods(MethodInfo[] methods, out MethodInfo[] forwardM
154154
List<MethodInfo>reverseMethodsList=newList<MethodInfo>();
155155
foreach(varmethodinmethods)
156156
{
157-
if(IsForward(method))
157+
if(IsReverse(method))
158158
{
159-
forwardMethodsList.Add(method);
159+
reverseMethodsList.Add(method);
160160
}else
161161
{
162-
reverseMethodsList.Add(method);
162+
forwardMethodsList.Add(method);
163163
}
164164

165165
}

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp