<div dir="ltr">OK, I see. I will try to cover that part in an elegant way.</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jul 13, 2018 at 4:31 AM Dmitry V. Levin <<a href="mailto:ldv@altlinux.org">ldv@altlinux.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Jul 12, 2018 at 02:49:42PM +0800, Zhibin Li wrote:<br>
> * tests/fcntl-common.c: Include <assert.h>.<br>
> (fcntl_cmd_check): New structure.<br>
> (print_retval_flags, test_other_set_cmd, test_other_get_cmd,<br>
> print_flags_getfd, print_flags_getsig, print_flags_getlease,<br>
> test_fcntl_others): New functions.<br>
> (main): Use test_fcntl_others.<br>
<br>
Applied, thanks.  This increased coverage of fcntl.c, but a few cases<br>
in print_fcntl() are still not covered by any tests.<br>
<br>
<br>
-- <br>
ldv<br>
</blockquote></div>