[issue7818] Improve set().test_c_api(): don't expect a set(abc), modify the content

2010-03-12 Thread STINNER Victor
STINNER Victor victor.stin...@haypocalc.com added the comment: Commited: r78886 (trunk), r7 (py3k). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue7818 ___

[issue7818] Improve set().test_c_api(): don't expect a set(abc), modify the content

2010-03-12 Thread STINNER Victor
Changes by STINNER Victor victor.stin...@haypocalc.com: -- resolution: - fixed status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue7818 ___

[issue7818] Improve set().test_c_api(): don't expect a set(abc), modify the content

2010-01-31 Thread STINNER Victor
STINNER Victor victor.stin...@haypocalc.com added the comment: The method was supposed to be for internal use only I found this bug by fuzzing. it may be worthwhile to build-out the test to accept many different possible inputs (...) Yeah, but I just want to avoid an assertion error :-)

[issue7818] Improve set().test_c_api(): don't expect a set(abc), modify the content

2010-01-30 Thread STINNER Victor
New submission from STINNER Victor victor.stin...@haypocalc.com: test_c_api() method of a set() (only defined in pydebug mode) suppose that the set content is abc. It causes assertion error if the method is called in a set different than set(abc). My patch modifies the set content at the

[issue7818] Improve set().test_c_api(): don't expect a set(abc), modify the content

2010-01-30 Thread STINNER Victor
Changes by STINNER Victor victor.stin...@haypocalc.com: -- components: +Interpreter Core type: - crash versions: +Python 2.7, Python 3.2 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue7818

[issue7818] Improve set().test_c_api(): don't expect a set(abc), modify the content

2010-01-30 Thread Benjamin Peterson
Changes by Benjamin Peterson benja...@python.org: -- assignee: - rhettinger nosy: +rhettinger ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue7818 ___

[issue7818] Improve set().test_c_api(): don't expect a set(abc), modify the content

2010-01-30 Thread Raymond Hettinger
Raymond Hettinger rhettin...@users.sourceforge.net added the comment: Will think about this. The method was supposed to be for internal use only. May just rename it to _test_c_api() and add a docstring noting that it is only defined for abc. That being said, it may be worthwhile to