python / cpython

The Python programming language
https://www.python.org/
Other
60.24k stars 29.15k forks source link

put back _Length functions for binary compatibility #32627

Closed Yhg1s closed 23 years ago

Yhg1s commented 23 years ago
BPO 400903
Nosy @malemburg, @Yhg1s
Files
  • None: None
  • 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: ```python assignee = None closed_at = created_at = labels = [] title = 'put back _Length functions for binary compatibility' updated_at = user = 'https://github.com/Yhg1s' ``` bugs.python.org fields: ```python activity = actor = 'twouters' assignee = 'none' closed = True closed_date = None closer = None components = ['None'] creation = creator = 'twouters' dependencies = [] files = ['2633'] hgrepos = [] issue_num = 400903 keywords = ['patch'] message_count = 3.0 messages = ['33419', '33420', '33421'] nosy_count = 2.0 nosy_names = ['lemburg', 'twouters'] pr_nums = [] priority = 'normal' resolution = None stage = None status = 'closed' superseder = None type = None url = 'https://bugs.python.org/issue400903' versions = [] ```

    Yhg1s commented 23 years ago
    Yhg1s commented 23 years ago

    Realized that myself last night, but I was too sleepy to fix it. This patch does.

    malemburg commented 23 years ago

    The patch looks OK, but I would like it to use the same technique as the code in ceval.c for PyEval_CallObject, that is: extensions compiled against the new code should see the APIs as macros while the old APIs are still available as tiny functions (just like in your patch).

    Could you update the patch accordingly ?