OK from manpages (don't forget to bump .Dd when committing).
Thanks for implementing this feature.

Thanks (I'll tell that to my optometrist ;-)).

Found another typo...

Link to zshexpn with zsh-specific chars.

Looks good in general, only minor nits found. Thanks for that work.

Good to go. Don't forget to bump the .Dd.

Comment about caret escaping in a popular shell...

OK from manpages.

Looks good now, ready to commit (and MFC later)! Thank you.

A comment about a sentence fragment. The rest looks good.
Do you know about textproc/igor? You can run it over the man page and it will give you some additional style tips and warnings. It will definitely complain about the .Dd not being set to today. For each content change that is relevant enough (like this one), we bump this to the date when the commit happens (i.e. when you get the approval).
Additionally, you can check the man page with "mandoc -Tlint" to see if there are some tags out of place or something.

OK from manpages, with the usual hint to bump the .Dd when the commit happens.

In D24916#548579, @jilles wrote:

Although there is nothing wrong with this per se, applications can deal with this problem more easily using getline(3) (which is also standard).

OK from manpages. Sorry it took a while to review.

OK from manpages.

Good catch, I'll commit the fix.
Thank you!

OK from manpages.

Thank you, Daniel. I think it is ready to commit now and will take care of it!

OK from manpages.

Good to go for the man page side of things.

In D24829#546556, @freqlabs wrote:

In D24829#546555, @debdrup wrote:

For what it's worth (not much, since I'm not yet a doc commiter), the manual page looks good.

Channelling my inner bcr, I must remember to bump the date before committing!

OK from manpages.

Comment about the BUGS section.

In D24635#542261, @gbergling_gmail.com wrote:

In D24635#542252, @bcr wrote:

Did that grammar fix come from upstream? If not, we should report it there and then get the updated version.

I send schwarze@openbsd.org an e-mail and reported the grammar mistake. I think that should work.

@0mp: Do you think we should keep the .Tn?
I lean towards committing the change as it is and handle the man page macros in a follow-up commit.

Looks good on the man page side of things, after removing the one extra "the" I found. Thanks for writing the man page!

OK from manpages.

OK from manpages.

Any updates on this? I think after make the changes commented above, it can be committed.

OK from manpages.

OK for the man page part of the change.

Approved!

I think that's a valid reason and I'm OK with the change. Thank you, Sergio.

I'll commit a slightly updated version soon. Thank you for the update.

Looks good. I'm taking care of the commit. I saw the review just now.

Is this related to https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=241683 ?

OK from manpages.

Looks good to me, so I'll go ahead and commit those. Thank you, Gordon!

Man page comments

Add Mariusz for Casper/Capsicum review. Not too deep, just a general "do the dates/releases make sense".

OK, I think this is good to have. I'll get it commited. Thank you!

I'll get this committed. Thanks!

OK from docs.

Looks good, I'll get this committed. Thank you for these changes!

I'm OK with it.

Add more man page folks for help.

Did that grammar fix come from upstream? If not, we should report it there and then get the updated version.

Great, thanks for these changes. I'll get them committed.

Looks good to me.

OK from manpages.

I'm OK with the change itself. Just want to make sure than @imp is fine with the content removal (it's available in the repo if someone needs it).

In D24617#541952, @lwhsu wrote:

Don't forget .Dd :-)

Looks good to to me. Thank you!

You can check the man page using textproc/igor and "mandoc -Tlint".

The man page looks good. Don't forget to bump the .Dd when committing.