-
-
Notifications
You must be signed in to change notification settings - Fork 32k
Porting argclinic socket.__init__ from Cinder. #92210
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
Comments
Coverage of argument clinic is not a goal. As for performance, the overhead of a slightly slower path to calling There's nothing wrong with improving the module as a whole to use argument clinic (it's nice to maintain less argument parsing code, while getting speedups for free when argument clinic improves), but the savings available from switching from using |
Thanks @MojoVampire I have revised my issue from your comment. |
Co-authored-by: Łukasz Langa <[email protected]> Co-authored-by: Erlend Egeberg Aasland <[email protected]> Co-authored-by: Erlend Egeberg Aasland <[email protected]> Co-authored-by: Jelle Zijlstra <[email protected]>
Thanks for your patch! |
Enhancement of socket.init to use argument clinic
In cinder this enhancement that is relatively easy involves adding argument clinic to socket.init . The primary goal will make this much easier to maintain.
The text was updated successfully, but these errors were encountered: