socket.htons uses unnecessary METH_VARARGS

[wrongnull]

Feature or enhancement

Proposal:

By replacing calling convention to METH_O it will run faster

Has this already been discussed elsewhere?

This is a minor feature, which does not need previous discussion elsewhere

Links to previous discussion of this feature:

No response

Linked PRs

• gh-111662: Update socket module to use AC for optimizing performance #111661

[serhiy-storchaka]

socket.ntohs() also can be optimized.

There is a larger patch in #64373 that converts the socket module to Argument Clinic. It would automatically apply such optimization for these functions and for many other.

[wrongnull]

socket.ntohs() also can be optimized.

There is a larger patch in #64373 that converts the socket module to Argument Clinic. It would automatically apply such optimization for these functions and for many other.

The main idea of this issue is replacing calling convention to METH_O when needed. I think the clinic argument is optional here

[wrongnull]

@serhiy-storchaka moreover, I don't think this patch is actual since there are now get_module_state function calls

[corona10]

socket.ntohs() also can be optimized.
There is a larger patch in #64373 that converts the socket module to Argument > Clinic. It would automatically apply such optimization for these functions and for many other.

#111661 updated several methods to use AC tool instead of manual parsing.
If someone wants to update other things too, please open a new issue.