➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author vstinner
Recipients pitrou, serhiy.storchaka, vstinner, yselivanov
Date 2016-02-05.17:43:06
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1454694194.19.0.667828891375.issue26298@psf.upfronthosting.co.za>
In-reply-to
Content 
Attached patch splits the huge "switch (opcode)" of ceval.c into smaller ceval_xxx.h files. New files:

   93 Python/ceval_stack.h
  142 Python/ceval_condjump.h
  155 Python/ceval_misc.h
  162 Python/ceval_fast.h
  180 Python/ceval_module.h
  238 Python/ceval_ctx.h
  249 Python/ceval_func.h
  262 Python/ceval_iter.h
  268 Python/ceval_build.h
  384 Python/ceval_number.h

Maybe we can put more files per .h file, maybe less. I don't really care.

It will allow to keep the code readable even with new optimizations like the issue #21955.

What do you think?
History
Date User Action Args
2016-02-05 17:43:15vstinnersetrecipients: + vstinner, pitrou, serhiy.storchaka, yselivanov
2016-02-05 17:43:14vstinnersetmessageid: <1454694194.19.0.667828891375.issue26298@psf.upfronthosting.co.za>
2016-02-05 17:43:14vstinnerlinkissue26298 messages
2016-02-05 17:43:13vstinnercreate