[lxml-dev] Test failures on Windows

[Stefan Behnel]

Hi Sidnei,

Sidnei da Silva wrote:
> | | You said that your patch fixes the problem, are you certain about that?
> | | Because calling xmlCtxtReset() should do exactly that (and a lot more) and it
> | | doesn't seem to solve the problem - according to the buildbot.
> 
> I see that you only call xmlClearContext if spaceTab is not
> NULL. Maybe that's the issue.

No, I also tried initialising spaceTab by hand to always call reset(). No
difference. So this is definitely not the problem. And it really makes me
wonder how your patch can work if xmlClearParserCtxt() does not, because your
code snippet is straight in there and I can't see a way it should not get
executed if clear() is called.

AFAICT, with the call to clear(), all input streams and memory resources
should get freed after parsing, so that's all I wanted. And still the problem
of a file descriptor staying open remains. I'll wait for the next buildbot run
to see, but if that fails, I'll really get clueless...

Note, BTW, that the error occurs in a finally block, so maybe it's already
shadowing an exception?

Stefan