[bylins]

Хвост из #4083, третья и последняя часть. #4086 убирает причину порчи списка, #4084 страхует цикл, а здесь закрывается сама ситуация, из которой всё выросло.

Что не так

Команда «следовать» ищет цель через FindCharInRoom и на NPC не проверяет — за мобом следовать можно. А MakeHorse делает скакуна последователем всадника:

void mount::MakeHorse(CharData *horse, CharData *ch) {
	AFF_FLAGS(horse).set(EAffect::kHorse);
	follow::AddFollower(ch, horse);

Если всадник в этот момент сам следует за этим мобом, выходит кольцо: set_master его отклоняет, и седловка проходит наполовину — сообщение «Вы оседлали ...» уже выведено, аффект kHorse выставлен, а подчинения нет.

Так и начиналось зависание из issue:

19:18:10  <Морхель> [след гон]
19:22:12  <Морхель> [оседлать гон]
19:22:12  попытка сделать цикл: лидер [Морхель], его текущий лидер [костяная гончая]
19:25:42  Морхель has rented → DieFollower → вечный цикл

Гончая — мир-моб #5902 из зоны 59 с флагом kMounting, её описание прямо предлагает оседлать; хозяина у неё на тот момент не было.

Правка

MakeHorse сперва снимает всадника со следования за этим мобом, и только потом седлает.

Правка в MakeHorse, а не в do_horsetake, чтобы накрыть и передачу скакуна (do_givehorse → MakeHorse(horse, victim)): принимающий точно так же мог за ним следовать.

StopFollower для всадника-игрока вернуть true не может — извлечение там только в ветке для NPC с kCorpse, — поэтому результат не проверяется.

Проверка

Сборка -Dbuild_profile=release -Dbuild_tests=true без предупреждений, ./tests/tests — 796 passed, 3 skipped (те же, что и до правки).

🤖 Generated with Claude Code