In D37930#861889, @jlduran_gmail.com wrote:

Also failing this test:

% ping -c1 192.0.2.1
PING 192.0.2.1 (192.0.2.1): 56 data bytes
Request timeout for icmp_seq 0
Request timeout for icmp_seq 0

--- 192.0.2.1 ping statistics ---
1 packets transmitted, 0 packets received, 100.0% packet loss

• sbin/ping: Do not show "Request timeout" when -q is specified
• sbin/ping: Fix the timeout message when -c or -l are specified

In D37493#856006, @t_uemura_macome.co.jp wrote:

• Many changes pointed to by hrs.
• Style(9) changes, some pointed to by takawata.

I put my comments inline. Many of them are style-related, but please fix the memory leak of the sc->res and sc->action queue, at least.

Looks good to me. Thanks for your hard work!

• Please update stf(4) manual page, too.
• I would suggest disabling 6to4 functionality when a braddr is configured. A mixed configuration of 6to4 and 6rd is quite confusing when seeing a result of ifconfig(8) command-line output because the braddr is a parameter for the interface, not a single 6rd domain configuration. I believe no one (except for me) uses the both at the same time.

If you need a second pair of eyes, I say this change looks good.

Replacements by sed can break literal "include" in the adoc source files. Is it always safe?

Looks good to me.