[igt-dev] [PATCH i-g-t 4/7] kms_flip: Remove unreachable condition in wait_for_events
Petri Latvala
petri.latvala at intel.com
Tue Mar 5 10:39:11 UTC 2019
On Mon, Mar 04, 2019 at 12:30:55PM -0300, Rodrigo Siqueira wrote:
> In the function wait_for_events() has a double check in the select()
> function return as described below:
>
> igt_assert_f(ret >= 0,
> "select error (errno %i)\n", errno);
> igt_assert_f(ret > 0,
> "select timed out or error (ret %d)\n", ret);
>
> Note that the second assert condition will not be reached because of the
> first assertion. This commit removes the code duplication and update the
> error message.
It will be reached if ret equals 0.
If select() returns 0, errno is unspecified, the main reason for the
separation.
--
Petri Latvala
More information about the igt-dev
mailing list