Proposal: allow std::byteswap from C++23

[David Benjamin]

This is hopefully a pretty easy one: C++23 adds std::byteswap. We have base::ByteSwap. Let's use the standard spelling!

https://chromium.googlesource.com/chromium/src/+/main/styleguide/c++/c++-features.md#std_byteswap-tbd

There were few enough uses that I was able to just migrate the lot of it in one go:

https://chromium-review.googlesource.com/c/chromium/src/+/7397193

David

[Mark Mentovai]

Non-controversial and LGTM!

--
You received this message because you are subscribed to the Google Groups "cxx" group.
To unsubscribe from this group and stop receiving emails from it, send an email to cxx+uns...@chromium.org.
To view this discussion visit https://groups.google.com/a/chromium.org/d/msgid/cxx/CAF8qwaA99yMX4biHqLxQQdEy_KTyt-dqRRcioUXX3hC%2B90QXFQ%40mail.gmail.com.

[Elly]

lgtm :)

-- elly

To view this discussion visit https://groups.google.com/a/chromium.org/d/msgid/cxx/CAHk1GDyQs0M-qgbo4zwhsFM73jj8MF3-TDyGFzs9B5Lv9_n0og%40mail.gmail.com.

[Avi Drissman]

Yes, let's do this.

To view this discussion visit https://groups.google.com/a/chromium.org/d/msgid/cxx/CADpj5oyobYh2mHkpWc8B1KWacCQyYsyAUeu8CRxJS4OTG34auw%40mail.gmail.com.

[Nico Weber]

I went ahead and lg'd the CL, since this looks very uncontroversial :)

To view this discussion visit https://groups.google.com/a/chromium.org/d/msgid/cxx/CACWgwAbVn2jtMmrE2tW4kdmHZXQV%2BvgG8JqSJKCncriVPs1_bQ%40mail.gmail.com.