Re: Fix Win32 file handle leak in translate-test.exe?

> >In the deltification-in-Apache case, it is both the only way and the
> >documented way to do cleanup after sending a response to a client. I
> >don't like it there either, but it was the best available option.
> >That's not the case with file handles.
> Exactly.
I don't get it: you depend on pool cleanup for closing the files yourself.
This contradicts what you have said until now. The only difference is that you
introduced a new pool to guarantee the files have been cleaned up at a
certain moment.

> And I'm totally against "fixing" this bug by changing the test. My patch
> fixes the cause, and I vote we include it in 0.35.0.

I didn't mean we have to change the tests. Never intended to declare the
tests wrong even.
Anyway; I'll stop about it now.

The fix is one of many possible but since it works I'm +1 on the patch and
including it in 0.35.0.

bye,

Erik.

-- 
+++ GMX - die erste Adresse für Mail, Message, More +++
Neu: Preissenkung für MMS und FreeMMS! http://www.gmx.net
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org