Skip to content
Snippets Groups Projects
Commit e98438bd authored by Nat Goodspeed's avatar Nat Goodspeed
Browse files

Fix subtle bug in ReadPipeImpl: wouldn't tolerate multiple instances.

That is, trying to instantiate a ReadPipeImpl while another already existed
would throw an LLEventPump::DupPumpName exception. Fortunately this behavior
is easily bypassed.
parent f52cf4be
No related branches found
No related tags found
No related merge requests found
...@@ -209,7 +209,7 @@ class ReadPipeImpl: public LLProcess::ReadPipe ...@@ -209,7 +209,7 @@ class ReadPipeImpl: public LLProcess::ReadPipe
mPipe(pipe), mPipe(pipe),
// Essential to initialize our std::istream with our special streambuf! // Essential to initialize our std::istream with our special streambuf!
mStream(&mStreambuf), mStream(&mStreambuf),
mPump("ReadPipe"), mPump("ReadPipe", true), // tweak name as needed to avoid collisions
// use funky syntax to call max() to avoid blighted max() macros // use funky syntax to call max() to avoid blighted max() macros
mLimit(npos) mLimit(npos)
{ {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment