Skip to content

Conversation

@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Dec 18, 2018

Use "ll" instead of the nonstandard "q".
(cherry picked from commit 062cbb6)

Co-authored-by: Zackery Spytz zspytz@gmail.com

https://bugs.python.org/issue10320

…yCArg_repr(). (pythonGH-10853)

Use "ll" instead of the nonstandard "q".
(cherry picked from commit 062cbb6)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM: previous backport already approved, see PR #10858.

@miss-islington
Copy link
Contributor Author

@ZackerySpytz and @serhiy-storchaka: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit 53e2248 into python:2.7 Dec 18, 2018
@miss-islington miss-islington deleted the backport-062cbb6-2.7 branch December 18, 2018 23:51
@ZackerySpytz
Copy link
Contributor

@vstinner No, ll can't be used in 2.7 (it's C99).

@vstinner
Copy link
Member

@vstinner No, ll can't be used in 2.7 (it's C99).

Oh something strange happened: PR #10858 has been approved, PR created by a bot, but the PR uses PY_FORMAT_LONG_LONG.

Oh, now I see @serhiy-storchaka's "Update callproc.c" commit in the other PR.

@serhiy-storchaka, @ZackerySpytz: Do you want to work on a PR to fix this mistake?

@ZackerySpytz
Copy link
Contributor

@vstinner Yes, I'll create a PR.

@ZackerySpytz
Copy link
Contributor

@vstinner I've created #11230.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants