Skip to content

Commit

Permalink
fix: store cache entry instead of icon in title_screens (ss220clu…
Browse files Browse the repository at this point in the history
…b#813)

## Что этот PR делает

Храним референс на файл в кеше, а не создаем иконку

## Почему это хорошо для игры

Не создаем лишние обьекты.

## Изображения изменений
Нет.

## Тестирование
Запустил игру, проверил - лобби скрин отображается

## Changelog

:cl:
fix: Храним референс на файл лобби скрина в кеше, а не создаем иконку.
Возможно пофиксит проблему с черным экраном
/:cl:

<!-- Оба :cl:'а должны быть на месте, что-бы чейнджлог работал! Вы
можете написать свой ник справа от первого :cl:, если хотите. Иначе
будет использован ваш ник на ГитХабе. -->
<!-- Вы можете использовать несколько записей с одинаковым префиксом
(Они используются только для иконки в игре) и удалить ненужные. Помните,
что чейнджлог должен быть понятен обычным игроком. -->
<!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы
можете исключить всю секцию. -->
  • Loading branch information
Gaxeer authored Dec 3, 2023
1 parent 85de728 commit 7ab4473
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions modular_ss220/title_screen/code/title_screen_subsystem.dm
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
#define TITLE_SCREENS_LOCATION "config/title_screens/images/"

/datum/controller/subsystem/title
flags = SS_NO_FIRE
init_order = INIT_ORDER_TITLE
Expand All @@ -18,18 +20,15 @@
title_html = file2text("config/title_html.txt")

var/list/local_title_screens = list()
for(var/screen in flist("config/title_screens/images/"))
for(var/screen in flist(TITLE_SCREENS_LOCATION))
var/list/screen_name_parts = splittext(screen, "+")
if((LAZYLEN(screen_name_parts) == 1 && (screen_name_parts[1] != "exclude" && screen_name_parts[1] != "blank.png")))
message_admins("Screen: [screen]")
local_title_screens += screen

for(var/title_screen in local_title_screens)
var/file_path = "config/title_screens/images/[title_screen]"
var/file_path = "[TITLE_SCREENS_LOCATION][title_screen]"
ASSERT(fexists(file_path))

var/icon/title2use = new(fcopy_rsc(file_path))
title_screens += title2use
title_screens += fcopy_rsc(file_path)

change_title_screen()

Expand Down Expand Up @@ -66,3 +65,5 @@
current_title_screen = DEFAULT_TITLE_SCREEN_IMAGE

show_title_screen()

#undef TITLE_SCREENS_LOCATION

0 comments on commit 7ab4473

Please sign in to comment.