killall(1): add EXAMPLES to manpage

[fernape]

Summary

killall(1) man page lacked EXAMPLES. Add some of them showing typical usage

Test Plan

1. Apply patch
2. man ./killall.1

[fernape]

Remove blank like after EXAMPLES section

[bjk]

Please remember to bump .Dd for the final version

killall/killall.1

149 ↗

(On Diff #72244)

I think .Dv would be appropriate for the plaintext signal names.

154 ↗

(On Diff #72244)

The Va macro can be used for variable names, though ${USER} is more of an expression than a variable name, so one could argue for it being .Li instead

[fernape]

Addressing comments by bcr@

• Use .Dv for signals
• Use .Va for ${USER} variable

[fernape]

killall/killall.1

149 ↗

(On Diff #72244)

Corrected here and in other places around the diff.

[bjk]

Thanks for the updates; they look good.
Just one more thing (stray .Ed) that I missed last time, sorry.
mandoc -Tlint can help find this sort of thing in the future.

killall/killall.1

160 ↗

(On Diff #72402)

It looks like there are a couple of .Ed macros at the end of regular paragraphs like this.
.Ed is only needed to close a matching .Bd section, around the actual example command-lines.

167 ↗

(On Diff #72402)

here too

192 ↗

(On Diff #72402)

Since "vim" and "vimdiff" are just examples of things that match "vim*", I don't feel a particular need to put extra markup (e.g., .Xr) on them

[fernape]

• Remove Ed without matching Bd (requested by bjk@)
• Removing Pp before Bd (reported by mandoc -Tlint)
• Remove extra white spaces at the end of the line (reported by mandoc -Tlint)

[fernape]

In D25002#552126, @bjk wrote:

Thanks for the updates; they look good.
Just one more thing (stray .Ed) that I missed last time, sorry.
mandoc -Tlint can help find this sort of thing in the future.

Thanks for the time spent reviewing this. I addressed also a couple of issues the linter spotted.

[bjk]

Looks good, thanks for the final updates!