[issue10246] uu.encode fd leak if arguments are filenames

Brian Brazil added the comment: The garbage collector should take care of the vast majority of these, it's only bugs in the C like issue 10253 that I'd worry about. -- ___ Python tracker _

[issue10246] uu.encode fd leak if arguments are filenames

Brett Cannon added the comment: On Sat, Oct 30, 2010 at 06:04, Antoine Pitrou wrote: > > Antoine Pitrou added the comment: > > Committed in r85975 (3.2). I guess we'll do a big svnmerge to other branches > later. Or not at all. I honestly have not been worrying about backporting since these

[issue10246] uu.encode fd leak if arguments are filenames

Antoine Pitrou added the comment: Committed in r85975 (3.2). I guess we'll do a big svnmerge to other branches later. -- resolution: -> fixed stage: -> committed/rejected status: open -> closed versions: +Python 3.2 -Python 3.3 ___ Python tracker

[issue10246] uu.encode fd leak if arguments are filenames

Antoine Pitrou added the comment: > How does v2 look? Nice, thank you! -- ___ Python tracker ___ ___ Python-bugs-list mailing list U

[issue10246] uu.encode fd leak if arguments are filenames

Brian Brazil added the comment: How does v2 look? -- Added file: http://bugs.python.org/file19426/uu_fd_leak_v2.patch ___ Python tracker ___

[issue10246] uu.encode fd leak if arguments are filenames

Antoine Pitrou added the comment: I think there should be a try..finally block so that those files get closed even when there's an error in-between. -- nosy: +pitrou ___ Python tracker ___

[issue10246] uu.encode fd leak if arguments are filenames

New submission from Brian Brazil : Please see attached patch, I'm not sure if this is the cleanest way to fix this. This also fixes the resource warnings in the test. -- components: Library (Lib) files: uu_fd_leak.patch keywords: patch messages: 119975 nosy: bbrazil, brett.cannon priori