Looks good. I say, check it in.
I can't argue with that <wink>.
Any thoughts on where this affects docs?
The only clear places I find are NEWS and C API's PyImport_ExecCodeModule docs. The latter doesn't mention sys.modules now, so I'll add stuff and note 2.4 changes.