Additional speedup (unsure about locals)

This I'm a little less sure of. I believe this doesn't modify global namespace, but put this in a separate commit since it changes the strategy. If OK, speed up versus previous commit, is 1.09, bringing total speedup to 1.37 vs main.

Author: ssweber

Lib/dataclasses.py

@@ -466,19 +466,16 @@ def _create_fn_def(name, args, body, *, locals=None, return_type=MISSING):
  return (name, txt, locals)
 
 def _exec_fn_defs(fn_defs, globals=None):
- # Free variables in exec are resolved in the global namespace.
- # The global namespace we have is user-provided, so we can't modify it for
- # our purposes. So we put the things we need into locals and introduce a
- # scope to allow the function we're creating to close over them.
- locals_dict = {k: v for _, _, locals_ in fn_defs 
- for k, v in locals_.items()}
- local_vars = ', '.join(locals_dict.keys())
- fn_names = ", ".join(name for name, _, _ in fn_defs)
- txt = "\n".join(f" {txt}" for _, txt, _ in fn_defs)
- txt = f"def __create_fn__({local_vars}):\n{txt}\n return {fn_names}"
- ns = {}
- exec(txt, globals, ns)
- return ns['__create_fn__'](**locals_dict)
+ def inner_executor():
+ txt = "\n".join(txt for _, txt, _ in fn_defs)
+ locals_dict = {k: v for _, _, locals_ in fn_defs 
+ for k, v in locals_.items()}
+ ns = {}
+ ns.update(locals_dict)
+ exec(txt, globals, ns)
+ return (ns[name] for name, _, _ in fn_defs)
+ return inner_executor()
+
 
 def _field_assign(frozen, name, value, self_name):
  # If we're a frozen class, then assign to our fields in __init__