8000 gh-81057: Move Signal-Related Globals to _PyRuntimeState by ericsnowcurrently · Pull Request #100085 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-81057: Move Signal-Related Globals to _PyRuntimeState #100085

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
merged 30 commits into from
Dec 12, 2022
Merged

gh-81057: Move Signal-Related Globals to _PyRuntimeState #100085

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
6dcffac
Add struct _signals_runtime_state.
ericsnowcurrently Dec 6, 2022
5e49464
Move signalmodule.c:Handlers to _PyRuntimeState.
ericsnowcurrently Dec 6, 2022
07332af
Move signalmodule.c:wakeup to _PyRuntimeState.
ericsnowcurrently Dec 6, 2022
1b5ec60
Move signalmodule.c:is_tripped to _PyRuntimeState.
ericsnowcurrently Dec 6, 2022
280f6f4
Move signalmodule.c:signal_global_state to _PyRuntimeState.
ericsnowcurrently Dec 7, 2022
fd6a2de
Fix includes (e.g. for Windows) in pycore_signal.h.
ericsnowcurrently Dec 8, 2022
410190b
Drop the socketmodule.h include.
ericsnowcurrently Dec 8, 2022
1fc3dce
Fix the include for SOCKET on Windows.
ericsnowcurrently Dec 8, 2022
d4f05e4
Fix formatting.
ericsnowcurrently Dec 8, 2022
1ddece1
Fix order of includes on WINDOWS.
ericsnowcurrently Dec 8, 2022
9592a8b
Fix includes on Windows.
ericsnowcurrently Dec 9, 2022
cc43787
Fix INVALID_FD on Windows.
ericsnowcurrently Dec 9, 2022
0164014
Revert "Fix includes on Windows."
ericsnowcurrently Dec 9, 2022
1d8d9e2
Include the headers conditionally.
ericsnowcurrently Dec 9, 2022
555f727
Fail instead of falling back to includes.
ericsnowcurrently Dec 9, 2022
57dc4a8
Reach for a sweet spot.
ericsnowcurrently Dec 10, 2022
08799ad
Pull in HANDLE.
ericsnowcurrently Dec 10, 2022
3bad4d6
Drop an unnecessary ifdef.
ericsnowcurrently Dec 12, 2022
3624161
Force the ifdef checks for the actual definition of _PyRuntime.
ericsnowcurrently Dec 12, 2022
2191c97
Drop the ifdef checks.
ericsnowcurrently Dec 12, 2022
76794fb
Change members/macros to fail (or warn) when used without the proper …
ericsnowcurrently Dec 12, 2022
c9aee62
Make sure the Windows includes happen early enough.
ericsnowcurrently Dec 12, 2022
ca33ad7
Move the includes up.
ericsnowcurrently Dec 12, 2022
7f35360
Check for _WINSOCKAPI_ instead of SOCKET.
ericsnowcurrently Dec 12, 2022
3d23778
Define a HANDLE macro where needed.
ericsnowcurrently Dec 12, 2022
73e3bc1
Use void* for the "fd" field.
ericsnowcurrently Dec 12, 2022
4195e62
Use void* for the "sigint_event" field.
ericsnowcurrently Dec 12, 2022
36cda7b
Use an int for wakeup.fd, instead of SOCKET.
ericsnowcurrently Dec 12, 2022
2a3bdf1
Drop the explicit includes.
ericsnowcurrently Dec 12, 2022
38596a0
Fix a warning.
ericsnowcurrently Dec 12, 2022
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
Next Next commit
Pull in HANDLE.
  • Loading branch information
@ericsnowcurrently
ericsnowcurrently committed Dec 12, 2022
commit 08799adab6b6f981b5ca4f66d4da3f14186b1d16
19 changes: 13 additions & 6 deletions Include/internal/pycore_signal.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,13 +13,13 @@ extern "C" {
#include "pycore_atomic.h" // _Py_atomic_address

#ifdef MS_WINDOWS
# ifndef SOCKET
# ifdef PYCORE_SIGNAL_REQUIRES_WINSOCK
# ifdef PYCORE_SIGNAL_WITH_PRE_INCLUDES
# ifndef SOCKET
# error "<winsock2.h> must be included before this header"
# endif
# endif
# ifndef HANDLE
# error "<windows.h> must be included before this header"
# ifndef HANDLE
# error "<windows.h> must be included before this header"
# endif
# endif
#endif
#include <signal.h> // NSIG
Expand Down Expand Up @@ -62,7 +62,8 @@ struct _signals_runtime_state {
# ifdef SOCKET
SOCKET fd;
# else
// <winsock2.h> wasn't included already, so we fake it.
// <winsock2.h> wasn't included already,
// we use something compatible with SOCKET.
int fd;
# endif
#elif defined(__VXWORKS__)
Expand All @@ -84,7 +85,13 @@ struct _signals_runtime_state {
PyObject *default_handler;
PyObject *ignore_handler;
#ifdef MS_WINDOWS
# ifdef HANDLE
HANDLE sigint_event;
# else
// <windows.h> wasn't included already,
// we use something compatible with HANDLE.
void *sigint_event;
# endif
#endif

/* True if the main interpreter thread exited due to an unhandled
Expand Down
4 changes: 2 additions & 2 deletions Modules/signalmodule.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@

#ifdef MS_WINDOWS
# if !defined(SOCKET) && defined(Py_INTERNAL_SIGNAL_H)
# error "pycore_signal.h included without PYCORE_SIGNAL_REQUIRES_WINSOCK"
# error "pycore_signal.h included without PYCORE_SIGNAL_WITH_PRE_INCLUDES"
# endif
# define _PYCORE_SIGNAL_REQUIRES_WINSOCK
# define PYCORE_SIGNAL_WITH_PRE_INCLUDES
#endif

#include "Python.h"
Expand Down
0