RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-05-31 14:28:12 +00:00
Code Issues Activity Actions

LibUnicode: Rename a special casing variable name in the UCD generator

Browse source 
This name will soon be a bit ambiguous with a similar case folding
variable name.
This commit is contained in:
Timothy Flynn 2023-01-17 08:11:06 -05:00 • committed by Linus Groh
parent 8d9fb898d7
commit 9226cf7272
1 changed files with 8 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

17
Meta/Lagom/Tools/CodeGenerators/LibUnicode/GenerateUnicodeData.cpp
View file

@ -112,7 +112,7 @@ struct UnicodeData {
Vector<SpecialCasing> special_casing; Vector<SpecialCasing> special_casing;
u32 code_points_with_special_casing { 0 }; u32 code_points_with_special_casing { 0 };
u32 largest_casing_transform_size { 0 }; u32 largest_special_casing_mapping_size { 0 };
u32 largest_special_casing_size { 0 }; u32 largest_special_casing_size { 0 };
Vector<DeprecatedString> conditions; Vector<DeprecatedString> conditions;
Vector<DeprecatedString> locales; Vector<DeprecatedString> locales;
@ -253,9 +253,9 @@ static ErrorOr<void> parse_special_casing(Core::Stream::BufferedFile& file, Unic
unicode_data.conditions.append(casing.condition); unicode_data.conditions.append(casing.condition);
} }
unicode_data.largest_casing_transform_size = max(unicode_data.largest_casing_transform_size, casing.lowercase_mapping.size()); unicode_data.largest_special_casing_mapping_size = max(unicode_data.largest_special_casing_mapping_size, casing.lowercase_mapping.size());
unicode_data.largest_casing_transform_size = max(unicode_data.largest_casing_transform_size, casing.titlecase_mapping.size()); unicode_data.largest_special_casing_mapping_size = max(unicode_data.largest_special_casing_mapping_size, casing.titlecase_mapping.size());
unicode_data.largest_casing_transform_size = max(unicode_data.largest_casing_transform_size, casing.uppercase_mapping.size()); unicode_data.largest_special_casing_mapping_size = max(unicode_data.largest_special_casing_mapping_size, casing.uppercase_mapping.size());
unicode_data.special_casing.append(move(casing)); unicode_data.special_casing.append(move(casing));
} }
@ -660,7 +660,6 @@ static ErrorOr<void> parse_unicode_data(Core::Stream::BufferedFile& file, Unicod
} }
bool has_special_casing { false }; bool has_special_casing { false };
for (auto const& casing : unicode_data.special_casing) { for (auto const& casing : unicode_data.special_casing) {
if (casing.code_point == data.code_point) { if (casing.code_point == data.code_point) {
data.special_casing_indices.append(casing.index); data.special_casing_indices.append(casing.index);
@ -688,7 +687,7 @@ static ErrorOr<void> generate_unicode_data_header(Core::Stream::BufferedFile& fi
{ {
StringBuilder builder; StringBuilder builder;
SourceGenerator generator { builder }; SourceGenerator generator { builder };
generator.set("casing_transform_size", DeprecatedString::number(unicode_data.largest_casing_transform_size)); generator.set("special_casing_mapping_size", DeprecatedString::number(unicode_data.largest_special_casing_mapping_size));
auto generate_enum = [&](StringView name, StringView default_, Vector<DeprecatedString> values, Vector<Alias> aliases = {}) { auto generate_enum = [&](StringView name, StringView default_, Vector<DeprecatedString> values, Vector<Alias> aliases = {}) {
quick_sort(values); quick_sort(values);
@ -751,13 +750,13 @@ namespace Unicode {
struct SpecialCasing { struct SpecialCasing {
u32 code_point { 0 }; u32 code_point { 0 };
u32 lowercase_mapping[@casing_transform_size@]; u32 lowercase_mapping[@special_casing_mapping_size@];
u32 lowercase_mapping_size { 0 }; u32 lowercase_mapping_size { 0 };
u32 uppercase_mapping[@casing_transform_size@]; u32 uppercase_mapping[@special_casing_mapping_size@];
u32 uppercase_mapping_size { 0 }; u32 uppercase_mapping_size { 0 };
u32 titlecase_mapping[@casing_transform_size@]; u32 titlecase_mapping[@special_casing_mapping_size@];
u32 titlecase_mapping_size { 0 }; u32 titlecase_mapping_size { 0 };
Locale locale { Locale::None }; Locale locale { Locale::None };