Фикс взрывов (нюки), который (вероятно) не будет смержен #11074
Conversation
Changelog status: ✔️ |
| @@ -26,7 +26,7 @@ var/global/list/explosion_turfs = list() | |||
| playsound(epicenter, pick(SOUNDIN_EXPLOSION), VOL_EFFECTS_MASTER, null, FALSE, null, round(power,1) ) | |||
|
|
|||
| explosion_in_progress = 1 | |||
| explosion_turfs = list() | |||
| explosion_turfs// = list() | |||
There was a problem hiding this comment.
Это глобальная переменная, которая сбрасывалась каждым новым рекурсивным взрывом. Нюка взрывала станцию лишь до первого встречного аккумулятора.
| if(T) | ||
| T.explosion_spread(spread_power, direction) | ||
| T = get_step(src, turn(direction,90)) | ||
| T.explosion_spread(side_spread_power, turn(direction,90)) | ||
| if(T) | ||
| T.explosion_spread(side_spread_power, turn(direction,90)) | ||
| T = get_step(src, turn(direction,-90)) | ||
| T.explosion_spread(side_spread_power, turn(direction,90)) | ||
| if(T) | ||
| T.explosion_spread(side_spread_power, turn(direction,90)) |
There was a problem hiding this comment.
не обрабатывался пустой get_step на краях карты, из-за чего очередь тайлов вероятно могла собраться не полностью
|
Я боюсь это мержить - пофикшенная нюка по сути планомерно удаляет станцию тайл за тайлом. Это напрашивается на контроллер, и на тг даже есть готовый, но я не собирался делать что-либо кроме фикса. Кто же знал... |
|
Это изменение нам надо! Но очень долгие взрывы. Нам надо то что будет ускорять их. |
|
Закрою в пользу #11090 Еще одна проблема в ПРе - из-за глобальной explosion_turfs каждый новый рекурсивный взрыв начинает ветвление и обход всех тех же турфов, и станция в результате взрывается в 15 потоков. |
|
Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности: https://forum.taucetistation.org/t/novye-vzryvy-starye-vzryvy-bomby-i-rnd/40238/1 |
Описание изменений
Почему и что этот ПР улучшит
Авторство
Чеинжлог
🆑