Skip to content

bpo-1635741: Port resource extension module to multiphase initialization(PEP 489) #19252

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
vstinner merged 8 commits into from
Apr 2, 2020
Merged

bpo-1635741: Port resource extension module to multiphase initialization(PEP 489) #19252

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
274da20
Port resource extension module to multiphase initialization(PEP 489)
shihai1991 Mar 31, 2020
6b46c86
update code
shihai1991 Mar 31, 2020
90504f7
fix some potential refleaks
shihai1991 Apr 1, 2020
7a82976
Simplify the code
shihai1991 Apr 2, 2020
a0dcd68
remove file's ADD_INT macro
shihai1991 Apr 2, 2020
7add9ba
Use PyModule_AddIntConstant to replace PyModule_AddIntMacro
shihai1991 Apr 2, 2020
41d6bf8
move position of claim of v
shihai1991 Apr 2, 2020
6fdb19a
Fix style issues
shihai1991 Apr 2, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix style issues
  • Loading branch information
@shihai1991
shihai1991 committed Apr 2, 2020
commit 6fdb19aecf32b10d43aeb8778617afdabcbff8fc
5 changes: 2 additions & 3 deletions Modules/resource.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -340,7 +340,8 @@ resource_methods[] = {
/* Module initialization */


static int resource_exec(PyObject *module)
static int
resource_exec(PyObject *module)
{
#define ADD_INT(module, value) \
do { \
Expand Down Expand Up @@ -471,14 +472,12 @@ static int resource_exec(PyObject *module)
#endif

PyObject *v;

if (sizeof(RLIM_INFINITY) > sizeof(long)) {
Comment thread
shihai1991 marked this conversation as resolved.
v = PyLong_FromLongLong((long long) RLIM_INFINITY);
} else
{
v = PyLong_FromLong((long) RLIM_INFINITY);
}

if (!v) {
return -1;
}
Comment thread
shihai1991 marked this conversation as resolved.
Expand Down