Skip to content

localization basis - #98

Merged
littleboobs merged 2 commits into
ss220-space:masterfrom
Ksaikok46:lokalization
May 25, 2026
Merged

localization basis#98
littleboobs merged 2 commits into
ss220-space:masterfrom
Ksaikok46:lokalization

Conversation

@Ksaikok46

@Ksaikok46 Ksaikok46 commented May 25, 2026

Copy link
Copy Markdown
Collaborator

Информация о PR

Основные проки и дефайны локализации (мог что-то забыть)

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

Данный пулл-реквест внедряет комплексную систему локализации для русского языка, включая дефайны грамматических падежей, вспомогательные процедуры для склонения числительных и учета рода, а также механизм кеширования имен атомов через систему declent_ru. В ходе анализа были выявлены критические ошибки: риск бесконечного цикла в процедуре genderize_decode при отсутствии закрывающего тега, логическая ошибка в сопоставлении направлений из-за несовпадения регистра в text2dir_rus, а также неэффективное и деструктивное использование replacetext, которое может приводить к удалению символов * вне контекста разметки. Требуется доработка логики обработки строк для обеспечения стабильности и производительности сервера.

Comment thread code/__HELPERS/~nemesis_helpers/localization/localization.dm
Comment thread code/__HELPERS/~nemesis_helpers/localization/localization.dm Outdated
Comment thread code/__HELPERS/~nemesis_helpers/localization/localization.dm
@littleboobs
littleboobs merged commit f9a14bc into ss220-space:master May 25, 2026
25 checks passed
@Ksaikok46
Ksaikok46 deleted the lokalization branch May 25, 2026 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants