Skip to content
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.

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

bpo-38321: Fix _asynciomodule.c compiler warning #16493

Merged
merged 1 commit into from
Sep 30, 2019
Merged

bpo-38321: Fix _asynciomodule.c compiler warning #16493

merged 1 commit into from
Sep 30, 2019

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Sep 30, 2019

bpo-38248, bpo-38321: Fix warning:

modules\_asynciomodule.c(2667):
warning C4102: 'set_exception': unreferenced label

The related goto has been removed by
commit edad4d8.

https://bugs.python.org/issue38321

bpo-38248, bpo-38321: Fix warning:

    modules\_asynciomodule.c(2667):
    warning C4102: 'set_exception': unreferenced label

The related goto has been removed by
commit edad4d8.
@vstinner vstinner merged commit efe74b6 into python:master Sep 30, 2019
@vstinner vstinner deleted the fix_asyncio_warn branch September 30, 2019 14:41
vstinner added a commit that referenced this pull request Oct 1, 2019
* bpo-38321: Fix _asynciomodule.c compiler warning (GH-16493)

bpo-38248, bpo-38321: Fix warning:

    modules\_asynciomodule.c(2667):
    warning C4102: 'set_exception': unreferenced label

The related goto has been removed by
commit edad4d8.

(cherry picked from commit efe74b6)

* bpo-38321: Fix PyCStructUnionType_update_stgdict() warning (GH-16492)

bpo-22273, bpo-38321: Fix following warning:

    modules\_ctypes\stgdict.c(704):
    warning C4244: 'initializing': conversion from 'Py_ssize_t' to 'int', possible loss of data

(cherry picked from commit c9a413e)
jacobneiltaylor pushed a commit to jacobneiltaylor/cpython that referenced this pull request Dec 5, 2019
bpo-38248, bpo-38321: Fix warning:

    modules\_asynciomodule.c(2667):
    warning C4102: 'set_exception': unreferenced label

The related goto has been removed by
commit edad4d8.
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