In D16697#505077, @linimon wrote:

Is there any reason that this should not be committed at this point?

Stock unpatched binary was in $PATH and would get executed by the patched master process.
Apologies for the noise.

Unfortunately not

In D16991#362417, @ebrandi wrote:

My bad sevan, really sorry for my mistake - I forgot completely about webarchive :(

There is also an updated code snippet at the website https://web.archive.org/web/20031204230654/http://www.geocities.com:80/SiliconValley/Code/4704/uliscanc.txt

The link you are proposing is to a different paper from the original. The original is still available via archive.org.
https://web.archive.org/web/20011115002319/http://www.geocities.com/SiliconValley/Code/4704/universal.pdf

Thank you for the patch

1. looked at the raw diff
2. checked the build status in this review
3. confirmed the zpool list output locally
4. approved :)

updated

Don't reword the title, it was ok to begin with.
Switch bash location from legacy to current path.
Drop superfluous sentence.

A new release of mandoc was posted last week, is this still valid?

As @kristof suggested.

In D15781#333724, @trasz wrote:

I hadn't ditched it, I've just moved towards the end of sentence.

another run.

reflow

Address the points.

Already in.

Change formatting.

Use application tag & capitalise Subversion.