WW #18

[Дмитрий Свиридкин]

стиль перемешался:

template<typename T>
constexpr bool hasFormat = hasFormatImpl<T>::is(nullptr);


template<typename T>
constexpr bool is_vector_v = is_vector<T>::value;

Сущности одинаковые, а конвенция именования разная.

Все-таки "паттерн матчинг" на уровне различных специализаций выглядит менее уродливо, чтом цепочка if constexpr. И придумывать магические числа для недостижимых веток не придется.

    char arr[size];
    auto [ptr, _] = std::to_chars(arr, arr + estimate_size() + 1, val);

Вы ж уже посчитали size. Зачем еще раз? to_chars(arr, arr + size. val)

if constexpr (std::is_same_v<bool, typename T::value_type>) {
     bool b = val[i];
     FormatHelper<typename T::value_type> helper(b);
     helper.append_to(s);
} else {
     FormatHelper<typename T::value_type> helper(val[i]);
     helper.append_to(s);
}

Можно вызвать append_to в одном выражении с вызовом конструктора и bool не успеет умереть. И if не нужен.

---

7 + 1.5