[C++17 DIS comment 014] [alg.reverse] reverse has two Requires: clauses

[zygoloid]

Figure out what happened here: how did we end up with two Requires: clauses that say the same thing in different ways? Then fix.

[cpplearner]

The first one was introduced in cf099ae, second one in 5c0f40c.

[jwakely]

Thanks, it looks like the merge of cf099ae to reorder the elements restored the old form of the Requires: instead of moving the new form that had just been edited by 5c0f40c

So the fix is to removethe first one, and move the second one up to that place.

[jwakely]

Reopening, because this was only fixed on master. The commit will need to be cherry-picked and the conflict resolved for the c++17 branch.

[tkoeppe]

Commit in master: d9075c8

[zygoloid]

Looks good, resolution: ACCEPTED WITH MODIFICATIONS. Existing p1 deleted, p3 moved to p1.

[zygoloid]

Merged to c++17 as 89801f1.