[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