A blast from the past: r15048 from way back removed the wrong boolean parameter

causing the message to be sent with a timeout of "true" (getting converted to 1)
instead of the intended 0 meaning no/infinite timeout. This caused the message
sending to be aborted due to the timeout if it was blocking on a full port for
example. Since the return value is never checked noone noticed.
It's possible that this was the cause of some lost input messages (mouse,
keyboard) when the system was under heavy enough load for either the port heap
to be exhausted (unlikely) or the input_server <-> app_server port to run full
(quite possible).


git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@42271 a95241bf-73f2-0310-859d-f6bbb57e9c96
This commit is contained in:
Michael Lotz 2011-06-20 22:24:36 +00:00
parent f092b1f782
commit b617e8daff

View File

@ -1795,7 +1795,7 @@ InputServer::_DispatchEvent(BMessage* event)
BMessenger reply;
BMessage::Private messagePrivate(event);
return messagePrivate.SendMessage(fAppServerPort, -1, 0, true, 0, reply);
return messagePrivate.SendMessage(fAppServerPort, -1, 0, 0, false, reply);
}