refactor(utf8): убрать запись через перекодировку (#3826)

[kvirund]

Пункты 1 и 2 из #3826. Данные на диске — UTF-8, движок держит текст в UTF-8, переводить при
записи нечего.

Что убрано

Последняя запись в KOI8-R. craft.cpp заявлял в XML-декларации encoding="koi8-r"
и гнал байты через native_text::write_file. Теперь и декларация, и байты — UTF-8, как
у мира, cfg, state и userdata.

Сами функции. to_disk (вместе с предохранителем внутри неё), перекодирующий write_file
и мёртвая from_utf8 — ноль вызовов. write_file_native (3 вызова) переименован обратно
в write_file: отдельное имя было нужно, только пока рядом жил перекодирующий двойник.

Пароли: перекодировка осталась, но названа честно

password.cpp и accounts.cpp переведены на явный to_koi8. Перекодировка там нужна, но она
про формат сохранённых хэшей, а не про кодировку диска: хэши посчитаны по кои-восьмым байтам,
и сменить это можно только перехэшированием при успешном входе. Под именем to_disk это читалось
как граница диска, которой больше нет, — и напрашивалось на переиспользование.

Тест HashIsTakenOverTheOnDiskForm переименован и переписан под ту же мысль: свойство проверяется
то же самое (два разных набора байт с одной кои-восьмой формой дают один хэш), но перестало
называться «дисковой формой».

Что осталось

Граница чтения (from_disk_line / from_disk_text / read_data_file) — на месте: файл,
написанный до миграции, ещё может попасться. Её снятие вместе с исходящим предохранителем —
следующий шаг #3826, и вот он уже точка невозврата.

Тесты границы записи заменены тестом границы чтения — проверяет разделитель и его
идемпотентность. Заодно поправлены комментарии, ссылавшиеся на исчезнувшую функцию,
и шапка native_text.h, где диск всё ещё числился кои-восьмым.

Сборка без предупреждений, 691 тест проходит.

[bylins]

Посмотрел ветку, пока она висит в draft. Слил её с нынешним master во временной ветке: конфликтов нет, собирается без предупреждений, 722 теста проходят (в master 723 — здесь два теста границы записи заменены одним тестом границы чтения).

По сути правка верная. to_disk для валидного UTF-8 просто вызывает to_koi8, так что замена в паролях на явный to_koi8 поведение не меняет, зато честнее называет: это формат старых хэшей, а не кодировка диска. Экспорт крафта теперь пишет encoding="utf-8" и UTF-8-байты, а старые выгрузки с объявлением koi8-r по-прежнему читаются — pugixml смотрит на объявление.

Два замечания.

1. Уходит предохранитель на записи. Вместе с to_disk исчезает её SYSERR «сюда пришли не те байты», который ловил пропущенную границу чтения. Сама граница чтения при этом остаётся, и в комментариях правки прямо сказано, что файл, написанный до миграции, ещё может попасться. Значит состояние, от которого предохранитель защищал, всё ещё возможно, а сигнала о нём больше не будет: кои-восьмые байты молча уедут в файл. Предлагаю оставить проверку внутри write_file — она недорогая, разбор строки идёт со скоростью порядка 477 МБ/с, а на записи файлов это тем более незаметно.

2. Мёртвая строка в glory_const.cpp:860. Там doc.append_attribute("encoding") = "koi8-r", но у xml_document атрибутов нет: в готовом lib/userdata/glory_const.xml стоит просто <?xml version="1.0"?>. Раз этот PR убирает последнее объявление koi8-r в записи (в craft.cpp), логично убрать заодно и эту строку.

Всё остальное чисто — из draft можно выводить.

[bylins]

Поправил по своему же разбору, раз добро получено:

• предохранитель с записи не потерян — проверка на не-UTF-8 переехала из to_disk в write_file. Граница чтения остаётся, файл до миграции ещё может попасться, так что сигнал о пропущенной границе нужен; байты по-прежнему пишутся как есть, ломаться нечему;
• убрана мёртвая строка doc.append_attribute("encoding") = "koi8-r" в glory_const.cpp: у xml_document атрибутов нет, в готовом файле стоит просто <?xml version="1.0"?>;
• в ветку влит master — она отставала на 219 коммитов.

Сборка чистая, 722 теста проходят. PR всё ещё в draft — снимать статус вам.