refactor: слой 1 -- "кто", setall и разбор аргументов (#3807)

[bylins]

Слой 1 из #3807: две команды и разбор аргументов, на который они обе опираются. Собрано из #3809, #3811 и #3812 — по коммиту на тему.

1. кто — глобальные буферы

Команда собирала вывод в buf, buf1 и arg (54 обращения на 360 строк). Буферы общие на весь процесс: вызов чужой функции посреди сборки строки затирает накопленное, а переполнение ловится сторожевым байтом уже постфактум.

Теперь разбор идёт через utils::ExtractFirstArgument, строка игрока и счётчики собираются fmt::format в std::string. sprintf, strcat и фиксированных буферов в файле не осталось.

Заодно кто боги снова работает:

if (!str_cmp(arg, "боги") && strlen(arg) == 4)

В KOI8-R «боги» — 4 байта, в UTF-8 — 8, поэтому вторая половина условия перестала выполняться и слово уходило в поиск по имени. Проверка длины тут не нужна: str_cmp сравнивает строки целиком.

И ширина колонки в кто -s. Имя сперва оборачивали в цвет, потом дополняли до тридцати знаков — а цветовой код это семь невидимых символов, сброс ещё семь, и fmt считает их наравне с буквами:

было : видимых знаков в колонке = 16
стало: видимых знаков в колонке = 30

2. setall — глобальные буферы и пропавшее сообщение

Сообщения бога собирались в buf1/buf2, разбор — в buf/buf1/buf2 через three_arguments.

В ветке смены e-mail у офлайнового бессмертного:

if (GetRealLevel(vict) >= kLvlGod) {
    it->second->out += buf1;      // buf1 здесь никто не заполнял

buf1 очищается в начале итерации и остаётся пустым — бог не получал ни строчки о том, почему персонаж пропущен, хотя во всех соседних ветках там стоит «Персонаж %s бессмертный!». Ровно тот класс ошибок, ради которого затеян слой 1: с общим буфером «забыл заполнить» и «кто-то заполнил раньше» выглядят в коде одинаково.

3. ExtractFirstArgument — доделка

Обе команды опираются на этот разбор, а он был неточен. Это моя же правка c0675a229 (март 2026): чинил алиасинг, закрыл симптом, вокруг не посмотрел.

• разделителем был только пробел, а one_argument режет по a_isspace. "frozen\t30 спам" давало слово frozen\t30. Функция разбирает dgaffect прямо из текста триггера, так что табуляция достижима;
• остаток начинался с пробелов — substr(pos + 1) пропускал ровно один, и каждый вызывающий дочищал сам;
• комментарий врал («если нет пробелов то строки пустые») и называл функцию аналогом one_argument, хотя тот вдобавок понижает регистр и пропускает служебные слова (in from with the on at to). Это разные задачи, но при переносе кода с half_chop про регистр надо помнить — иначе почта в setall перестанет находиться, она сверяется с player_table подстрокой.

У half_chop в заголовке появилось предупреждение: остаток он копирует с оглядкой на kMaxStringLength, а one_argument длину не проверяет вовсе — буфер меньшего размера передавать нельзя.

Проверено отдельно: алиасинг ExtractFirstArgument(str, str) (ради него был фикс #3037) сохранён — во всех ветках слово копируется до записи в remains; есть тест на разбор в цикле через одну переменную. Многобайтные буквы не рвутся: набор разделителей чисто ASCII, продолжающие байты UTF-8 всегда ≥ 0x80.

Одно изменение поведения: значение с ведущими пробелами теперь отдаётся без них. Задевает хранение переменных триггеров (dg_scripts.cpp:3517/3557), где строка перед разбором и так проходит Split с TrimLeft — пробелы после имени переменной были артефактом формата, а не данными.

Проверено

Сборка чистая, 681 тест зелёный (673 + восемь новых на ExtractFirstArgument).

На живом: кто, кто -s, кто -n имя, кто -c профессия, кто боги, кто 20-30; setall <почта> frozen|email|passwd|hell ... — и отдельно случай с бессмертным в списке, ради пропавшего сообщения.

Закрывает #3809, #3811, #3812.

🤖 Generated with Claude Code

https://claude.ai/code/session_01XUwDWDnYdXrJdvjDVd36QH