Skip to content

Commit d3f8428

Browse files
Apply suggestions from code review
Co-authored-by: Adam Turner <[email protected]>
1 parent 3c66cf5 commit d3f8428

File tree

1 file changed

+17
-5
lines changed

1 file changed

+17
-5
lines changed

Modules/_decimal/_decimal.c

Lines changed: 17 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4346,7 +4346,9 @@ nm_##MPDFUNC(PyObject *self, PyObject *other) \
43464346
return result; \
43474347
}
43484348

4349-
/* Boolean function without a context arg. */
4349+
/* Boolean function without a context arg.
4350+
Argument Clinic provides PyObject *self
4351+
*/
43504352
#define Dec_BoolFunc(MPDFUNC) \
43514353
{ \
43524354
return MPDFUNC(MPD(self)) ? incr_true() : incr_false(); \
@@ -4419,7 +4421,10 @@ nm_##MPDFUNC(PyObject *self, PyObject *other) \
44194421

44204422
/* Binary function with an optional context arg. Actual MPDFUNC does
44214423
NOT take a context. The context is used to record InvalidOperation
4422-
if the second operand cannot be converted exactly. */
4424+
if the second operand cannot be converted exactly.
4425+
4426+
Argument Clinic provides PyObject *self, PyObject *other, PyObject *context
4427+
*/
44234428
#define Dec_BinaryFuncVA_NO_CTX(MPDFUNC) \
44244429
{ \
44254430
PyObject *a, *b; \
@@ -6047,7 +6052,9 @@ static PyType_Spec dec_spec = {
60476052
return ret; \
60486053
}
60496054

6050-
/* Unary context method. */
6055+
/* Unary context method.
6056+
Argument Clinic provides PyObject *context, PyObject *x
6057+
*/
60516058
#define DecCtx_UnaryFunc(MPDFUNC) \
60526059
{ \
60536060
PyObject *result, *a; \
@@ -6071,7 +6078,9 @@ static PyType_Spec dec_spec = {
60716078
return result; \
60726079
}
60736080

6074-
/* Binary context method. */
6081+
/* Binary context method.
6082+
Argument Clinic provides PyObject *context, PyObject *v, PyObject *w
6083+
*/
60756084
#define DecCtx_BinaryFunc(MPDFUNC) \
60766085
{ \
60776086
PyObject *a, *b; \
@@ -6130,7 +6139,10 @@ ctx_##MPDFUNC(PyObject *context, PyObject *args) \
61306139
return result; \
61316140
}
61326141

6133-
/* Ternary context method. */
6142+
/* Ternary context method.
6143+
Argument Clinic provides PyObject *context, PyObject *v, PyObject *w,
6144+
PyObject *x
6145+
*/
61346146
#define DecCtx_TernaryFunc(MPDFUNC) \
61356147
{ \
61366148
PyObject *a, *b, *c; \

0 commit comments

Comments
 (0)