<div dir="ltr"><div>On Wed, Jun 14, 2017 at 01:39:47PM +0300, Dmitry V. Levin wrote:</div><div>> On Wed, Jun 14, 2017 at 11:54:16AM +0800, JingPiao Chen wrote:</div><div>> > * tests/printstr.c: Add a case for room < DEFAULT_STRLEN + 1.</div><div>> </div><div>> According to the coverage statistics, this branch of code is already</div><div>> tested, but one more explicit test won't harm anyway.</div><div><br></div><div>I write this case because it make me understand printstr behavior clearer.</div><div>I did not know that before. Maybe help other understand it.</div><div><br></div><div>> </div><div>> > ---</div><div>> >  tests/printstr.c | 7 ++++++-</div><div>> >  1 file changed, 6 insertions(+), 1 deletion(-)</div><div>> > </div><div>> > diff --git a/tests/printstr.c b/tests/printstr.c</div><div>> > index 330234f..035ce9c 100644</div><div>> > --- a/tests/printstr.c</div><div>> > +++ b/tests/printstr.c</div><div>> > @@ -40,7 +40,7 @@ int</div><div>> >  main(void)</div><div>> >  {</div><div>> >  <span class="gmail-Apple-tab-span" style="white-space:pre">     </span>char *const buf = tail_alloc(DEFAULT_STRLEN + 1);</div><div>> > -<span class="gmail-Apple-tab-span" style="white-space:pre">   </span>const struct iovec io = {</div><div>> > +<span class="gmail-Apple-tab-span" style="white-space:pre">   </span>struct iovec io = {</div><div>> >  <span class="gmail-Apple-tab-span" style="white-space:pre">                </span>.iov_base = buf,</div><div>> >  <span class="gmail-Apple-tab-span" style="white-space:pre">           </span>.iov_len = -1</div><div>> >  <span class="gmail-Apple-tab-span" style="white-space:pre">      </span>};</div><div>> > @@ -64,6 +64,11 @@ main(void)</div><div>> >  <span class="gmail-Apple-tab-span" style="white-space:pre">   </span>tprintf("writev(-1, [{iov_base=\"\\0%*s\"..., iov_len=%lu}], 1)"</div><div>> >  <span class="gmail-Apple-tab-span" style="white-space:pre">               </span>" = %s\n", DEFAULT_STRLEN - 1, buf + 1, -1UL, sprintrc(rc));</div><div>> >  </div><div>> > +<span class="gmail-Apple-tab-span" style="white-space:pre">   </span>io.iov_base += 1;</div><div>> </div><div>> I'd change this to ++io.iov_base;</div><div>> </div><div>> > +<span class="gmail-Apple-tab-span" style="white-space:pre">        </span>rc = writev(-1, &io, 1);</div><div>> > +<span class="gmail-Apple-tab-span" style="white-space:pre">        </span>tprintf("writev(-1, [{iov_base=%p, iov_len=%lu}], 1)"</div><div>> > +<span class="gmail-Apple-tab-span" style="white-space:pre">             </span>" = %s\n", io.iov_base, -1UL, sprintrc(rc));</div><div>> </div><div>> I wouldn't split the first argument because it fits the line.</div><div>> </div><div>> > +</div><div>> >  <span class="gmail-Apple-tab-span" style="white-space:pre">    </span>tprintf("+++ exited with 0 +++\n");</div><div>> >  <span class="gmail-Apple-tab-span" style="white-space:pre">      </span>return 0;</div><div>> >  }</div><div>> </div><div>> No need to post v2, just keep in mind.</div><div><br></div><div>Thank you.</div><div><br></div><div>--</div><div>JingPiao Chen</div></div>