◐ Shell
reader mode source ↗
Skip to content

bpo-24037: Add Argument Clinic converter bool(accept={int}).#485

Merged
serhiy-storchaka merged 4 commits into
python:masterfrom
serhiy-storchaka:bpo-24037-clinic-boolint-converter
Mar 12, 2017
Merged

bpo-24037: Add Argument Clinic converter bool(accept={int}).#485
serhiy-storchaka merged 4 commits into
python:masterfrom
serhiy-storchaka:bpo-24037-clinic-boolint-converter

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Mar 5, 2017

Copy link
Copy Markdown
Member

It is used for semantical booleans using 'i' format unit. This allows to not repeat the default value for Python and C and will help in converting to bool in future.

@serhiy-storchaka serhiy-storchaka added the type-feature A feature request or enhancement label Mar 5, 2017
@mention-bot

mention-bot commented Mar 5, 2017

Copy link
Copy Markdown

@serhiy-storchaka, thanks for your PR! By analyzing the history of the files in this pull request, we identified @benjaminp, @ezio-melotti, @Haypo, @doerwalter and @loewis to be potential reviewers.

@serhiy-storchaka serhiy-storchaka merged commit 202fda5 into python:master Mar 12, 2017
@serhiy-storchaka serhiy-storchaka deleted the bpo-24037-clinic-boolint-converter branch March 12, 2017 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vadmium vadmium vadmium left review comments

@vstinner vstinner vstinner approved these changes

@taleinat taleinat Awaiting requested review from taleinat

@larryhastings larryhastings Awaiting requested review from larryhastings

+1 more reviewer

@duboviy duboviy duboviy approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

type-feature A feature request or enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@serhiy-storchaka @mention-bot @vstinner @vadmium @duboviy @the-knights-who-say-ni