diff options
author | bulatman <bulatman@yandex-team.com> | 2023-06-01 10:15:45 +0300 |
---|---|---|
committer | bulatman <bulatman@yandex-team.com> | 2023-06-01 10:15:45 +0300 |
commit | 4dea553457efda88fef237809afb5b9a95da7d41 (patch) | |
tree | 276164112112c63ee03db7d5b257900663d1908f /library/cpp/yt/misc/enum-inl.h | |
parent | 636774447d59eeee64e2f500f2c1816c53cb96c4 (diff) | |
download | ydb-4dea553457efda88fef237809afb5b9a95da7d41.tar.gz |
YT: Fix ParseEnum when custom domain names used
Diffstat (limited to 'library/cpp/yt/misc/enum-inl.h')
-rw-r--r-- | library/cpp/yt/misc/enum-inl.h | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/library/cpp/yt/misc/enum-inl.h b/library/cpp/yt/misc/enum-inl.h index 951e93c130..c7e0abdab2 100644 --- a/library/cpp/yt/misc/enum-inl.h +++ b/library/cpp/yt/misc/enum-inl.h @@ -60,6 +60,23 @@ constexpr bool CheckValuesUnique(const TValues& values) return true; } +template <typename TNames> +constexpr bool CheckDomainNames(const TNames& names) +{ + for (size_t i = 0; i < std::size(names); ++i) { + if (std::size(names[i]) == 0) { + return false; + } + for (size_t j = 1; j < std::size(names[i]); ++j) { + // If name does not start with a capital letter, all the others must be in lowercase. + if (('A' <= names[i][j] && names[i][j] <= 'Z') && ('A' > names[i][0] || names[i][0] > 'Z')) { + return false; + } + } + } + return true; +} + } // namespace NDetail //////////////////////////////////////////////////////////////////////////////// @@ -81,6 +98,8 @@ constexpr bool CheckValuesUnique(const TValues& values) static constexpr std::array<TStringBuf, DomainSize> Names{{ \ PP_FOR_EACH(ENUM__GET_DOMAIN_NAMES_ITEM, seq) \ }}; \ + static_assert(::NYT::NDetail::CheckDomainNames(Names), \ + "Enumeration " #enumType " contains names in wrong format"); \ static constexpr std::array<T, DomainSize> Values{{ \ PP_FOR_EACH(ENUM__GET_DOMAIN_VALUES_ITEM, seq) \ }}; \ |