Chromium Code Reviews
DescriptionFixes leak in Process.runSync().
This removes the call to _stdin._sink.destroy()
from my previous CL, which broke a test.
stdin is closed by the native implementation.
Closing again shouldn't cause a problem, but
it looks like destroy() attempts to do a lot
more than that.
OTOH adding the remove call is sufficient to
fix the leak, so let's just stick with that.
R=iposva@google.com
Committed: https://code.google.com/p/dart/source/detail?r=41395
Patch Set 1 #
Messages
Total messages: 4 (1 generated)
|
|||||||||||||||||||