1121080 - mfbt/Move.h comments describing perfect forwarding are incorrect

Status: RESOLVED FIXED

(Core :: MFBT, defect)

Description

[Jim Blandy :jimb]

The comments in mfbt/Move.h describing perfect forwarding incorrectly state the types to which the template type parameters are resolved.

I think the (soon-to-be) attached patch brings Move.h in line with the description here:

http://thbecker.net/articles/rvalue_references/section_08.html

and the description in the "Nitty Gritty Details" section of this page:

http://isocpp.org/blog/2012/11/universal-references-in-c11-scott-meyers

Comment 1

[Jim Blandy :jimb]

Attachment: Fix comments in Move.h explaining perfect forwarding.

Comment 2

[Jeff Walden [:Waldo]]

Comment on attachment 8548350 [details] [diff] [review]
Fix comments in Move.h explaining perfect forwarding.

Review of attachment 8548350 [details] [diff] [review]:
-----------------------------------------------------------------

Glorious.  We probably should introduce "forwarding reference" as a Thing in these docs somewhere:

http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2014/n4164.pdf

But one thing at a time, and I am not all that perturbed about the lack at present.

Comment 3

[Jim Blandy :jimb]

https://hg.mozilla.org/integration/mozilla-inbound/rev/74abc1cf0193

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/74abc1cf0193