Updates:
        Status: Done

Comment #9 on issue 1700 by pekka.klarck: Process: Executed process leave file handles open
http://code.google.com/p/robotframework/issues/detail?id=1700

This issue was updated by revision f5008a50a4ad.

Fixed the problem by changing `close_custom_streams` method to close
also standard streams. Obviously also dropped `custom` from the method
name. The implementation is a bit longer than necessary, but wanted to
make it as explicit as possible. For example, reading streams by just
accessing `self.stdout` or `self.stderr` seemed a bit magic.

We alraedy had tests for running multiple processes both using default
(i.e. standard) and custom streams. They only executed 42 processes
which wasn't enough to trigger the problem. Now they run 500 which
seems to be enough. Don't want to make that unnecessarily large number
to avoid increasing test execution time.

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--

--- You received this message because you are subscribed to the Google Groups "robotframework-commit" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to robotframework-commit+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to