[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: AW: Python bindings: Use of svn_swig_py_make_stream() - avoiding leaking fds

From: Nick Piper <nick.piper_at_logica.com>
Date: Wed, 11 May 2011 11:05:34 +0100

On 10/05/11 16:08, Bert Huijben wrote:

> The python bindings need it, so I would say: Add it there.

The attached patch fixes the FD leak entirely within the Python bindings.

> (And maybe make it easier to use from other bindings when it proves useful. That is why I suggested adding the function in svn_stream.h)

I'm not familiar enough with the use of streams in subversion to be able
to argue for any particular approach; so I hope that by keeping this
entirely to the Python bindings it may be committed.

I expect we'll use this patch (today's version) in our production
service very soon.


Sorry for this disclaimer:
Think green - keep it on the screen.
This e-mail and any attachment is for authorised use by the intended recipient(s) only. It may contain proprietary material, confidential information and/or be subject to legal privilege. It should not be copied, disclosed to, retained or used by, any other party. If you are not an intended recipient then please promptly delete this e-mail and any attachment and all copies and inform the sender. Thank you. 
Received on 2011-05-11 12:07:00 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.