| commit | b561a5fac7a20e2bbb26b690d2a34fe4b096a6b3 | [log] [tgz] |
|---|---|---|
| author | brettw <brettw@chromium.org> | Fri Mar 24 09:36:42 2017 -0700 |
| committer | Commit bot <commit-bot@chromium.org> | Fri Mar 24 16:36:42 2017 +0000 |
| tree | 88f548d72f4ff2466285f6a3d6ee257fcaac3220 | |
| parent | 33959291b6a2b12ac31233ed948df422b51e5b4b [diff] |
Enable noexcept on Windows, use for a few move constructors. MSVC complains if you use noexcept with no exception handling mode specified (as we do). This code disables the warning. noexcept on move constructors allows better optimizations in some cases. http://en.cppreference.com/w/cpp/language/noexcept_spec Updates a few common classes' move constructors to use this. Reland of https://codereview.chromium.org/2771643002 with landmine. There were some reports of PCH headers not getting updated and the warning change no taking effect. I investigated briefly but could not see why PCH dependencies are incorrect. Review-Url: https://codereview.chromium.org/2769283002 Cr-Original-Commit-Position: refs/heads/master@{#459455} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: f78cc27d15ccbd1fd0c5c93aa03b939cd89b9b89