Recommend noexcept for move constructors.

Recommend noexcept for move constructors.

Marking move constructors "noexcept" allows std::vector to use the move
constructors in container resizes. Otherwise it will copy which is not
expected by most callers.
http://en.cppreference.com/w/cpp/language/move_constructor

This also replaces some technically invalid characters with HTML entities.

Discussion:
https://groups.google.com/a/chromium.org/d/msg/cxx/ze4WJFg7RvU/Qhbb6DTrBQAJ

Review-Url: https://codereview.chromium.org/2782973002
Cr-Commit-Position: refs/heads/master@{#460530}
Committed: https://chromium.googlesource.com/chromium/src/+/9efba0cd0d357c433d878635e673d5de4a2f4be1

[brettw, 2017-03-29 19:36:24]

Description was changed from

==========
Recommend noexcept for move constructors.

Marking move constructors "noexcept" allows std::vector to use the move
constructors in container resizes. Otherwise it will copy which is not
expected by most callers.
http://en.cppreference.com/w/cpp/language/move_constructor

Discussion:
https://groups.google.com/a/chromium.org/d/msg/cxx/ze4WJFg7RvU/Qhbb6DTrBQAJ
==========

to

==========
Recommend noexcept for move constructors.

Marking move constructors "noexcept" allows std::vector to use the move
constructors in container resizes. Otherwise it will copy which is not
expected by most callers.
http://en.cppreference.com/w/cpp/language/move_constructor

This also replaces some technically invalid characters with HTML entities.

Discussion:
https://groups.google.com/a/chromium.org/d/msg/cxx/ze4WJFg7RvU/Qhbb6DTrBQAJ
==========

[brettw, 2017-03-29 19:36:24]

brettw@chromium.org changed reviewers:
+ jbroman@chromium.org

[jbroman, 2017-03-29 19:41:32]

This lgtm. Agreed that explicitly calling out why move constructors but not move
assignment operators makes the wording here a little long.

[danakj, 2017-03-29 19:45:00]

danakj@chromium.org changed reviewers:
+ danakj@chromium.org

[danakj, 2017-03-29 19:45:01]

LGTM too

[brettw, 2017-03-29 19:53:52]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2017-03-29 19:55:04]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-29 20:49:45]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490817232598880, "parent_rev":
"9752d65b1a7b678630b51d29017e268aacafa647", "commit_rev":
"9efba0cd0d357c433d878635e673d5de4a2f4be1"}

[commit-bot: I haz the power, 2017-03-29 20:50:32]

Description was changed from

==========
Recommend noexcept for move constructors.

Marking move constructors "noexcept" allows std::vector to use the move
constructors in container resizes. Otherwise it will copy which is not
expected by most callers.
http://en.cppreference.com/w/cpp/language/move_constructor

This also replaces some technically invalid characters with HTML entities.

Discussion:
https://groups.google.com/a/chromium.org/d/msg/cxx/ze4WJFg7RvU/Qhbb6DTrBQAJ
==========

to

==========
Recommend noexcept for move constructors.

Marking move constructors "noexcept" allows std::vector to use the move
constructors in container resizes. Otherwise it will copy which is not
expected by most callers.
http://en.cppreference.com/w/cpp/language/move_constructor

This also replaces some technically invalid characters with HTML entities.

Discussion:
https://groups.google.com/a/chromium.org/d/msg/cxx/ze4WJFg7RvU/Qhbb6DTrBQAJ

Review-Url: https://codereview.chromium.org/2782973002
Cr-Commit-Position: refs/heads/master@{#460530}
Committed:
https://chromium.googlesource.com/chromium/src/+/9efba0cd0d357c433d878635e673...
==========

[commit-bot: I haz the power, 2017-03-29 20:50:33]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/9efba0cd0d357c433d878635e673...