L120: Transition to Modern C++

[veblush]

No description provided.

[markdroth]

Thanks for doing this!

[markdroth]

Do we want to have to keep publishing new gRFCs every time we add support for a new version of C++, or do we just want to publish a gRFC that says "from now on, we'll follow the OSS foundational C++ support policy"?

[veblush]

It's a good point. Our plan is to keep up with the OSS foundational C++ support policy so we can have this last gRFC making it clear and skip new gRFC for the next bump.

[veblush]

I mentioned the future plan at the bottom. PTAL.

[markdroth]

If this is actually our plan going forward, then I think we should re-title this gRFC -- it's less about C++17 specifically than it is about the fact that we're now going to follow the OSS foundational C++ support guidelines.

Maybe this warrants a brief discussion at our Tuesday design review meeting?

[veblush]

That's a good point. I changed it and I'll bring this up in the next design meeting.

[eugeneo]

Are we going to allow, e.g., to std::string_view from absl::string_view? Should this gRFC mention this?

[veblush]

Are we going to allow, e.g., to std::string_view from absl::string_view? Should this gRFC mention this?

This topic is beyond the scope of this doc but that should be communicated well within the maintainers. Abseil by default makes absl::string_view an alias of std::string_view under C++17 but it still provides an opt-in build option to use their own implementation. Hence, to support this build option, gRPC will have to stick to absl::string_view until we stop supporting this option.

[veblush]

Are we going to allow, e.g., to std::string_view from absl::string_view? Should this gRFC mention this?

This topic is beyond the scope of this doc but that should be communicated well within the maintainers. Abseil by default makes absl::string_view an alias of std::string_view under C++17 but it still provides an opt-in build option to use their own implementation. Hence, to support this build option, gRPC will have to stick to absl::string_view until we stop supporting this option.

grpc/grpc#37930 just got merged to prevent accidental use of std::string_view in the future.