Skip to content
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

Jump to bottom

kwdargs for compile/__import__ #42990

Closed
Yhg1s opened this issue Mar 7, 2006 · 4 comments
Closed

kwdargs for compile/__import__ #42990

Yhg1s opened this issue Mar 7, 2006 · 4 comments
Assignees
@birkenfeld
Labels
interpreter-core (Objects, Python, Grammar, and Parser dirs)

Comments

@Yhg1s
Copy link
Member

Yhg1s commented Mar 7, 2006

BPO 1444529
Nosy @loewis, @Yhg1s, @birkenfeld
Files
  • kwds.diff
    		•

    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:

    assignee = 'https://github.com/birkenfeld'
closed_at = <Date 2007-03-13.20:47:10.000>
created_at = <Date 2006-03-07.00:55:30.000>
labels = ['interpreter-core']
title = 'kwdargs for compile/__import__'
updated_at = <Date 2007-03-13.20:47:10.000>
user = 'https://github.com/Yhg1s'

    bugs.python.org fields:

    activity = <Date 2007-03-13.20:47:10.000>
actor = 'georg.brandl'
assignee = 'georg.brandl'
closed = True
closed_date = None
closer = None
components = ['Interpreter Core']
creation = <Date 2006-03-07.00:55:30.000>
creator = 'twouters'
dependencies = []
files = ['7056']
hgrepos = []
issue_num = 1444529
keywords = ['patch']
message_count = 4.0
messages = ['49681', '49682', '49683', '49684']
nosy_count = 4.0
nosy_names = ['loewis', 'twouters', 'nnorwitz', 'georg.brandl']
pr_nums = []
priority = 'normal'
resolution = 'accepted'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue1444529'
versions = ['Python 2.5']
    @Yhg1s
    Copy link
    Member Author

    Yhg1s commented Mar 7, 2006

    Keyword arguments for compile() and __import__(), as
    discussed at PyCon. There is a slight speed difference
    on my development system: __import__ is ~1% slower,
    compile() is ~1.5% faster. I blame both on caches,
    alignment and what not, so I bet they're totally
    different for everyone. I'd say the advantage of
    managing the growing set of arguments outweighs the
    speed impact (if any.)

    @Yhg1s Yhg1s closed this as completed Mar 7, 2006
    @Yhg1s Yhg1s added the interpreter-core (Objects, Python, Grammar, and Parser dirs) label Mar 7, 2006
    @Yhg1s Yhg1s closed this as completed Mar 7, 2006
    @Yhg1s Yhg1s added the interpreter-core (Objects, Python, Grammar, and Parser dirs) label Mar 7, 2006
    @nnorwitz
    Copy link
    Mannequin

    nnorwitz mannequin commented Mar 7, 2006

    Logged In: YES
    user_id=33168

    These look good to me. Thanks Thomas. I had started a patch
    for import. BTW I think the new __import__ needs to be
    documented in libfuncs.txt or somewhere like that. If you
    don't get to it, I'll find my change and update the docs.

    @loewis
    Copy link
    Mannequin

    loewis mannequin commented Mar 5, 2007

    Is anybody still working on this?

    @birkenfeld
    Copy link
    Member

    __import__ already accepted kwargs, committed the compile() part in rev. 54358.

    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Assignees

    @birkenfeld birkenfeld

    Labels
    interpreter-core (Objects, Python, Grammar, and Parser dirs)
    Projects
    None yet
    Milestone
    No milestone
    Development

    No branches or pull requests
    2 participants
    @birkenfeld @Yhg1s