-
-
Notifications
You must be signed in to change notification settings - Fork 29.4k
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
socket: finish constant to Enum/Flag conversion #72270
Comments
Split from bpo-23591. Add IntFlag constants for consistency with IntEnum constants. |
New changeset 4a027e55dae3 by Ethan Furman in branch 'default': |
socketmodule.c has more namespaces. Are you planning to wrap them, too? NETLINK_ (*) constant namespace has sub-namespaces, e,g ALG_SET_ and ALG_OP_ |
I don't have enough socket experience to know which constants should go into what groups and what those groups should be called. So far my involvement has been to switch other's patches to use Enum._convert instead of "globals().update(Enum.__members__)". (This last patch was from Serhiy.) I am not opposed, though. ;) If Larry is okay with adding those during beta 1 I can work on it, but there's no way I could get it done before tomorrow. |
Still need to update docs. |
You have to as Ned, not Larry. :) |
One could argue that it is a bug that the other constants are missing. Ethan, if you can work with Christian to get them in by b2, that would be acceptable. |
Thanks, Ned! |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: