`utils::ForwardLike` is not actually like `std::forward_like`

[ddvamp]

userver/universal/include/userver/utils/forward_like.hpp

Lines 13 to 21 in 525eba6

	// Analogue of std::forward_like from c++23.
	template <typename TOwner, typename TMember>
	decltype(auto) ForwardLike(TMember& member) {
	if constexpr (std::is_lvalue_reference_v<TOwner> || std::is_lvalue_reference_v<TMember>) {
	return member;
	} else {
	return std::move(member);
	}
	}

userver/universal/include/userver/utils/forward_like.hpp

Lines 23 to 27 in 525eba6

	// Analogue of std::forward_like from c++23.
	template <typename TOwner, typename TMember>
	decltype(auto) ForwardLike(const TMember& member) {
	return member;
	}

Данная реализация не является точным аналогом std::forward_like. Нужно либо привести правильную реализацию, либо отобразить факт различия в названии и/или описании.

[ddvamp]

Просмотрел коммит с добавлением этого кода, и возник ряд вопросов.

• Ошибочно заменён std::move на std::forward для правой ссылки (ссылка).
• Во множестве мест идиоматическая конструкция std::forward<T>(t).field заменена на использование ForwardLike, что снижает читаемость (пример).
• USERVER_DEFINE_STRUCT_SUBSET не работает с правыми ссылками. Не уверен, намеренное ли это поведение.
• Идиоматическое обращение с кортежом заменено на использование ForwardLike, что менее читаемо.