Author eli.bendersky
Recipients Arfrever, brett.cannon, eli.bendersky, eric.snow, ezio.melotti, pitrou
Date 2013-02-06.13:19:18
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <>
In-reply-to <>
On Wed, Feb 6, 2013 at 4:17 AM, Ezio Melotti <> wrote:

> Ezio Melotti added the comment:
> I'm still a bit skeptic about this.
> To summarize my position, I am:
> +0 about adding something like this to and use it for new
> tests;
> -1 about mass-rewriting the existing tests to use this (in particular for
> test_json);
> +1 about simplifying the API of import_fresh_module so that it returns
> both the modules and doesn't require the fresh/blocked args;

I share Ezio's sentiment regarding the new proposal(s). As for
import_fresh_module, if it's only ever used to do what we describe here,
then I propose to rename it to something more descriptive and indeed make
it just return both modules. If it's being used for other stuff, we can add
another function and rewrite all the tests to use that.

py_module, c_module = import_python_module_and_accelerator('module',
Date User Action Args
2013-02-06 13:19:18eli.benderskysetrecipients: + eli.bendersky, brett.cannon, pitrou, ezio.melotti, Arfrever, eric.snow
2013-02-06 13:19:18eli.benderskylinkissue17037 messages
2013-02-06 13:19:18eli.benderskycreate