Skip to content

chore(olc): не выпускать макрос ZCMD за пределы своего файла - #3805

Merged
bylins merged 1 commit into
masterfrom
chore/zcmd-macro-leak
Aug 29, 2026
Merged

chore(olc): не выпускать макрос ZCMD за пределы своего файла#3805
bylins merged 1 commit into
masterfrom
chore/zcmd-macro-leak

Conversation

@bylins

@bylins bylins commented Aug 29, 2026

Copy link
Copy Markdown
Owner

Сборка ругалась при каждом прогоне:

src/engine/olc/redit.cpp:100: warning: "ZCMD" redefined
src/engine/olc/oedit.cpp:252: note: this is the location of the previous definition

В unity-сборке файлы склеиваются в одну единицу трансляции, поэтому определение из oedit.cpp доживало до redit.cpp и переопределялось там.

Что сделано

  • oedit — макрос убран совсем: он был объявлен и ни разу не использован, единственное упоминание в файле — само определение;
  • medit — макрос используется в medit_save_internally, поэтому после функции добавлен #undef, как уже сделано в zedit и redit.

Проверено

Сборка чистая — предупреждений не осталось ни одного, 673 теста зелёные.

🤖 Generated with Claude Code

https://claude.ai/code/session_01XUwDWDnYdXrJdvjDVd36QH

Сборка ругалась: redit.cpp:100: "ZCMD" redefined. В unity-сборке файлы
склеиваются в одну единицу трансляции, а определение из oedit.cpp никто не
закрывал -- оно доживало до redit.cpp и переопределялось там.

В oedit макрос вообще не использовался: объявлен и ни разу не помянут, только
мешал. Убран.

В medit используется в medit_save_internally -- туда добавлен #undef сразу за
функцией, как уже сделано в zedit и redit.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XUwDWDnYdXrJdvjDVd36QH
@bylins
bylins merged commit 8215b7d into master Aug 29, 2026
20 checks passed
@bylins
bylins deleted the chore/zcmd-macro-leak branch August 29, 2026 12:53
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