Re: svn commit: r1156375 - /subversion/branches/fs-py/subversion/python/svn/fs.py

On Wed, Aug 10, 2011 at 17:26, <hwright_at_apache.org> wrote:
> Author: hwright
> Date: Wed Aug 10 21:26:36 2011
> New Revision: 1156375
>
> URL: http://svn.apache.org/viewvc?rev=1156375&view=rev
> Log:
> On the fs-py branch:
> Followup to r1156347: use a bit cleaner syntax to automagically close files.
>
> * subversion/python/svn/fs.py
>  (FS.set_uuid, FS._open_fs): Use the with statement.

And now where did your .close() call go? You're still relying on 'f'
going out of scope to close. IOW, right back to the zero refcount
algorithm. And in which case, the original construction is much
cleaner than all this "with" gunk.

???
Received on 2011-08-10 23:36:23 CEST