Patch: When demandimport is disabled the 'test-profile' test fails.

Nicolas Dumazet nicdumz at gmail.com
Tue Jul 28 20:13:45 CDT 2009


2009/7/29 Alejandro Santos <alejolp at alejolp.com>:
> +  File "/tmp/hgtests.xStzl7/install/lib/python/mercurial/lsprof.py",
> line 88, in label
> +    for k, v in sys.modules.iteritems():
> +RuntimeError: dictionary changed size during iteration
> +--profile + output to file failed

I dont understand what causes this behavior in the loop body. Do you
understand the problem?


> -        for k, v in sys.modules.iteritems():
> +        for k, v in list(sys.modules.iteritems()):


We probably want
   for k, v in sys.modules.items():
here, don't we?

-- 
Nicolas Dumazet — NicDumZ [ nɪk.d̪ymz ]



More information about the Mercurial-devel mailing list