Commit 69112658 authored Aug 24, 2023 by Siarhei Vishniakou

Don't log when there are no more messages

Throughout input code, we check for presence of messages by looking at
the return code of various read functions. These are usually infinite
reads that get terminated when EWOULDBLOCK is received.

This is guaranteed to be hit every time (otherwise, we would get an
infinite loop). So in this CL, we remove the misleading log about
EWOULDBLOCK, which looks like this:

InputTransport: channel '...WindowInputTests$TestActivity (server)' publisher ~ receiveConsumerResponse: finished: seq=2966, handled=false
InputTransport: channel '...WindowInputTests$TestActivity (server)' publisher ~ receiveConsumerResponse: Unknown error -11

Bug: 282957027
Test: logcatcolor | grep -i input
Change-Id: Ie83fc1c9b69c02d8c48ebebaf4e57e0f76c9a3d9

parent e3ce4125

libs/input/InputTransport.cpp

+4 −2

Original line number	Diff line number	Diff line
		@@ -791,8 +791,10 @@ android::base::Result<InputPublisher::ConsumerResponse> InputPublisher::receiveC
		InputMessage msg;
		status_t result = mChannel->receiveMessage(&msg);
		if (result) {
		ALOGD_IF(debugTransportPublisher(), "channel '%s' publisher ~ %s: %s",
		mChannel->getName().c_str(), __func__, strerror(result));
		if (debugTransportPublisher() && result != WOULD_BLOCK) {
		LOG(INFO) << "channel '" << mChannel->getName() << "' publisher ~ " << __func__ << ": "
		<< strerror(result);
		}
		return android::base::Error(result);
		}
		if (msg.header.type == InputMessage::Type::FINISHED) {