Skip to content

bpo-29793: Convert some builtin types constructors to Argument Clinic.#615

Merged
serhiy-storchaka merged 1 commit into
python:masterfrom
serhiy-storchaka:bpo-29793-clinic-constructors
Mar 19, 2017
Merged

bpo-29793: Convert some builtin types constructors to Argument Clinic.#615
serhiy-storchaka merged 1 commit into
python:masterfrom
serhiy-storchaka:bpo-29793-clinic-constructors

Conversation

@serhiy-storchaka
Copy link
Copy Markdown
Member

Converted constructors are:

  • complex.__new__
  • float.__new__
  • function.__new__
  • int.__new__
  • mappingproxy.__new__
  • module.__init__
  • property.__init__
  • structseq.__new__

@serhiy-storchaka serhiy-storchaka added the type-feature A feature request or enhancement label Mar 11, 2017
@mention-bot
Copy link
Copy Markdown

@serhiy-storchaka, thanks for your PR! By analyzing the history of the files in this pull request, we identified @benjaminp, @tim-one, @mdickinson, @brettcannon and @tiran to be potential reviewers.

Comment thread Objects/descrobject.c

#include "clinic/descrobject.c.h"

PyTypeObject PyDictProxy_Type = {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since you're refactoring this piece of code anyways, maybe we can start using designated initializers?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't want to do this in this commit. This will be harder to review and will be easier to make an error.

@serhiy-storchaka serhiy-storchaka merged commit 18b250f into python:master Mar 19, 2017
@serhiy-storchaka serhiy-storchaka deleted the bpo-29793-clinic-constructors branch March 19, 2017 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-feature A feature request or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants