| commit | fb5850f410437e8baf6c4b8c43c4695835e95b6e | [log] [tgz] |
|---|---|---|
| author | Mike Klein <mtklein@google.com> | Mon Nov 09 15:50:37 2020 -0600 |
| committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | Tue Nov 10 14:47:36 2020 +0000 |
| tree | 06de071b643c98021fceae0a57e9aa3feb07cc30 | |
| parent | 0c0884991d53083c59ec4b9abca921f134544f71 [diff] |
replace std::aligned_storage There's no reason to use std::aligned_storage when it's simpler to use an array and alignas(). This way you don't have to remember whether the template arguments are size-then-align or align-then-size, you don't have to remember to use the _t variant or typename ... ::type, and there's no risk to forgetting the alignment parameter entirely. It doesn't look like this was deprecated, but I still think this paper makes good arguments for why we shouldn't use it: http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2019/p1413r1.pdf Bug: skia:10921 Change-Id: Ia64a2e43c4cba9b4d64138a7474e353a8eaf01a6 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/333258 Reviewed-by: Herb Derby <herb@google.com> Commit-Queue: Mike Klein <mtklein@google.com>