Skip to content

fix(sight): widen the vnum column to the allowed seven digits - #3786

Merged
bylins merged 1 commit into
masterfrom
fix/wide-vnum-column
Aug 28, 2026
Merged

fix(sight): widen the vnum column to the allowed seven digits#3786
bylins merged 1 commit into
masterfrom
fix/wide-vnum-column

Conversation

@bylins

@bylins bylins commented Aug 28, 2026

Copy link
Copy Markdown
Owner

Внум печатался в поле шириной пять знаков, а разрешено семь — kMaxProtoNumber = 9999999 (boot_constants.h:9). Предметы и мобы из зон от тысячной в пять знаков не влезают, и строка съезжает вправо:

<на плечах>                 [97845] стеганый кафтан Чанфы (велик)
<в обеих руках>             [100109] вершитель судеб (велик двур) ..шумит! ..горит!

Зон с внумами от 100000 сейчас 39, только предметов в них 1026 — то есть колонка ломается регулярно, и дальше будет чаще.

Что сделано

Поле расширено до семи знаков в трёх местах sight.cpp: список предметов (:1294), список существ (:1890) и заголовок комнаты (:191). Пятизначные внумы теперь печатаются с двумя дополнительными пробелами слева — [ 3975] вместо [ 3975], зато колонка держится на любом внуме.

Видят это не только иммы: внум показывается всем, у кого включён EPrf::kRoomFlags, и любому игроку в тестовой зоне (InTestZone).

Команды богов (stat, vnum, show, users) печатают внум в своих форматах — их не трогал.

Проверено

Сборка чистая, 669 тестов зелёные.

🤖 Generated with Claude Code

https://claude.ai/code/session_01XUwDWDnYdXrJdvjDVd36QH

Внум печатался в поле шириной пять знаков, а разрешено семь
(kMaxProtoNumber = 9999999). Предметы и мобы из зон от тысячной уже не влезают,
и строка съезжает вправо, ломая колонку:

  <на плечах>      [97845] стеганый кафтан Чанфы (велик)
  <в обеих руках>  [100109] вершитель судеб (велик двур)

Таких зон сейчас 39, только предметов в них больше тысячи. Поле расширено до
семи знаков в трёх местах: предметы, мобы и заголовок комнаты. Видят это не
только иммы -- внумы показываются и игрокам в тестовой зоне.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XUwDWDnYdXrJdvjDVd36QH
@bylins
bylins merged commit 609e386 into master Aug 28, 2026
20 checks passed
@bylins
bylins deleted the fix/wide-vnum-column branch August 28, 2026 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant