[PATCH] initialise optional parameters in Python-C bindings

Alexander Bokovoy ab at samba.org
Thu Apr 12 07:12:02 UTC 2018

On to, 12 huhti 2018, Douglas Bagnall via samba-technical wrote:
> I noticed today how easy it is to leave a variable uninitialised using
> the PyArg_ParseTuple* functions with the "|" token, and how disguised
> this unsetness is hidden from both static checkers and people's eyes.
> The "|" means all the rest of the arguments are optional and the
> corresponding C variables won't be changed if they are omitted.
> I applied some grep and found the following.
RB+. Please push.

/ Alexander Bokovoy

More information about the samba-technical mailing list