[issue10257] Fix resource warnings in test_os

2010-10-30 Thread Brian Brazil
New submission from Brian Brazil brian.bra...@gmail.com: Please see attached. -- components: Tests files: test_os_fd_leak.patch keywords: patch messages: 120015 nosy: bbrazil priority: normal severity: normal status: open title: Fix resource warnings in test_os versions: Python 3.3

[issue10257] Fix resource warnings in test_os

2010-10-30 Thread Brian Brazil
Brian Brazil brian.bra...@gmail.com added the comment: I'm used to a slightly different style guide, v2 has the right indentation. -- Added file: http://bugs.python.org/file19438/test_os_fd_leak_v2.patch ___ Python tracker rep...@bugs.python.org

[issue10257] Fix resource warnings in test_os

2010-10-30 Thread Éric Araujo
Éric Araujo mer...@netwok.org added the comment: I don’t see any difference :) -- nosy: +eric.araujo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10257 ___

[issue10257] Fix resource warnings in test_os

2010-10-30 Thread Brian Curtin
Brian Curtin cur...@acm.org added the comment: Fixed in r85987. Made both places hunks of the patch use the context manager. -- assignee: - brian.curtin nosy: +brian.curtin resolution: - fixed stage: - committed/rejected type: - resource usage versions: +Python 3.2 -Python 3.3

[issue10257] Fix resource warnings in test_os

2010-10-30 Thread Brian Curtin
Changes by Brian Curtin cur...@acm.org: -- status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10257 ___ ___ Python-bugs-list