Странное определение operator bool()

[eoan-ermine]

userver/core/include/userver/server/http/form_data_arg.hpp

Lines 15 to 22 in cb8efc3

	struct FormDataArg {
	std::string_view value;
	std::string_view content_disposition;
	std::optional<std::string> filename;
	std::optional<std::string> default_charset;
	std::optional<std::string_view> content_type;
	explicit operator bool() const { return !!value.data(); }

Непонятное назначение двух инверсий. Ведь value.data(), указатель и сам скастуется в bool при необходимости. Это преобразование не считается чем-либо постыдным и не вызывает предупреждений ни у каких компиляторов.

Не уверен, что проброс value.data() это лучший способ реализовать operator bool() для данного типа, ведь поле value теоретически может быть пустой строкой:

form_data_arg.value = ""

И тогда operator bool() вернет true, хотя, возможно, должен вернуть false. Я недостаточно хорошо знаю код, чтобы понять, какая семантика требуется.

[apolukhin]

Выглядит подозрительно, скорее всего там должен быть !empty() . Перепроверим и поправим. В любом случае, спасибо за репорт!

А !! - это старый трюк чтобы вызвать преобразование к bool и при этом избежать warning от некоторых компиляторов и анализаторов

[apolukhin]

Удалили, чтобы поведение оператора не оказалось сюрпризом при использовании

[apolukhin]

Спасибо за багрепорт!