mirror of
https://github.com/Bubberstation/Bubberstation.git
synced 2026-07-13 00:55:20 +01:00
21b4095dfd
Upstream 04/17/2026 fixes https://github.com/Bubberstation/Bubberstation/issues/5549 --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: tgstation-ci[bot] <179393467+tgstation-ci[bot]@users.noreply.github.com> Co-authored-by: ArcaneMusic <41715314+ArcaneMusic@users.noreply.github.com> Co-authored-by: MrMelbert <51863163+MrMelbert@users.noreply.github.com> Co-authored-by: Rhials <28870487+Rhials@users.noreply.github.com> Co-authored-by: rageguy505 <54517726+rageguy505@users.noreply.github.com> Co-authored-by: LemonInTheDark <58055496+LemonInTheDark@users.noreply.github.com> Co-authored-by: John Willard <53777086+JohnFulpWillard@users.noreply.github.com> Co-authored-by: Aliceee2ch <160794176+Aliceee2ch@users.noreply.github.com> Co-authored-by: Time-Green <7501474+Time-Green@users.noreply.github.com> Co-authored-by: Tsar-Salat <62388554+Tsar-Salat@users.noreply.github.com> Co-authored-by: SmArtKar <44720187+SmArtKar@users.noreply.github.com> Co-authored-by: Maxipat <108554989+Maxipat112@users.noreply.github.com> Co-authored-by: SyncIt21 <110812394+SyncIt21@users.noreply.github.com> Co-authored-by: deltanedas <39013340+deltanedas@users.noreply.github.com> Co-authored-by: SimplyLogan <47579821+loganuk@users.noreply.github.com> Co-authored-by: loganuk <fakeemail123@aol.com> Co-authored-by: Leland Kemble <70413276+lelandkemble@users.noreply.github.com> Co-authored-by: FalloutFalcon <86381784+FalloutFalcon@users.noreply.github.com> Co-authored-by: Roxy <75404941+TealSeer@users.noreply.github.com> Co-authored-by: Lucy <lucy@absolucy.moe> Co-authored-by: siliconOpossum <138069572+siliconOpossum@users.noreply.github.com> Co-authored-by: Isratosh <Isratosh@hotmail.com> Co-authored-by: TheRyeGuyWhoWillNowDie <70169560+TheRyeGuyWhoWillNowDie@users.noreply.github.com> Co-authored-by: Neocloudy <88008002+Neocloudy@users.noreply.github.com> Co-authored-by: Alexander V. <volas@ya.ru> Co-authored-by: ElGitificador <168473461+ElGitificador@users.noreply.github.com> Co-authored-by: Twaticus <46540570+Twaticus@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Bloop <13398309+vinylspiders@users.noreply.github.com> Co-authored-by: Cameron Lennox <killer65311@gmail.com> Co-authored-by: Tim <timothymtorres@gmail.com> Co-authored-by: Iamgoofball <iamgoofball@gmail.com> Co-authored-by: Layzu666 <121319428+Layzu666@users.noreply.github.com> Co-authored-by: Arturlang <24881678+Arturlang@users.noreply.github.com> Co-authored-by: _0Steven <42909981+00-Steven@users.noreply.github.com> Co-authored-by: mrmanlikesbt <99309552+mrmanlikesbt@users.noreply.github.com> Co-authored-by: Ben10Omintrix <138636438+Ben10Omintrix@users.noreply.github.com> Co-authored-by: John F. Kennedy <54908920+MacaroniCritter@users.noreply.github.com> Co-authored-by: Cursor <102828457+theselfish@users.noreply.github.com> Co-authored-by: Josh <josh.adam.powell@gmail.com> Co-authored-by: Josh Powell <josh.powell@softwire.com> Co-authored-by: Yobrocharlie <Charliemiller5617@gmail.com> Co-authored-by: Hardly3D <66234359+Hardly3D@users.noreply.github.com> Co-authored-by: shayoki <96078776+shayoki@users.noreply.github.com> Co-authored-by: LT3 <83487515+lessthnthree@users.noreply.github.com>
73 lines
3.2 KiB
Plaintext
73 lines
3.2 KiB
Plaintext
#define ALLOWED_LOOSE_TURFS 100
|
|
/**
|
|
* Responsible for managing the sizes of area.contained_turfs and area.turfs_to_uncontain
|
|
* These lists do not check for duplicates, which is fine, but it also means they can balloon in size over time
|
|
* as a consequence of repeated changes in area in a space
|
|
* They additionally may not always resolve often enough to avoid memory leaks
|
|
* This is annoying, so lets keep an eye on them and cut them down to size if needed
|
|
*/
|
|
SUBSYSTEM_DEF(area_contents)
|
|
name = "Area Contents"
|
|
ss_flags = SS_NO_INIT
|
|
runlevels = RUNLEVEL_LOBBY|RUNLEVELS_DEFAULT
|
|
var/list/currentrun
|
|
var/list/area/marked_for_clearing = list()
|
|
|
|
/datum/controller/subsystem/area_contents/stat_entry(msg)
|
|
var/total_clearing_from = 0
|
|
var/total_to_clear = 0
|
|
for(var/area/to_clear as anything in marked_for_clearing)
|
|
for (var/area_zlevel in 1 to length(to_clear.turfs_to_uncontain_by_zlevel))
|
|
if (length(to_clear.turfs_to_uncontain_by_zlevel[area_zlevel]))
|
|
total_to_clear += length(to_clear.turfs_to_uncontain_by_zlevel[area_zlevel])
|
|
if (length(to_clear.turfs_by_zlevel) >= area_zlevel) //this should always be true, but stat_entry is no place for runtimes. fire() can handle that
|
|
total_clearing_from += length(to_clear.turfs_by_zlevel[area_zlevel])
|
|
msg = "\n A:[length(currentrun)] MR:[length(marked_for_clearing)] TC:[total_to_clear] CF:[total_clearing_from]"
|
|
return ..()
|
|
|
|
|
|
/datum/controller/subsystem/area_contents/fire(resumed)
|
|
if(!resumed)
|
|
currentrun = GLOB.areas.Copy()
|
|
|
|
while(length(currentrun))
|
|
var/area/test = currentrun[length(currentrun)]
|
|
for (var/area_zlevel in 1 to length(test.turfs_to_uncontain_by_zlevel))
|
|
if(length(test.turfs_to_uncontain_by_zlevel[area_zlevel]) > ALLOWED_LOOSE_TURFS)
|
|
marked_for_clearing |= test
|
|
break
|
|
currentrun.len--
|
|
if(MC_TICK_CHECK)
|
|
return
|
|
|
|
// Alright, if we've done a scan on all our areas, it's time to knock the existing ones down to size
|
|
while(length(marked_for_clearing))
|
|
var/area/clear = marked_for_clearing[length(marked_for_clearing)]
|
|
|
|
for (var/area_zlevel in 1 to length(clear.turfs_to_uncontain_by_zlevel))
|
|
if (!length(clear.turfs_to_uncontain_by_zlevel[area_zlevel]))
|
|
continue
|
|
if (length(clear.turfs_by_zlevel) < area_zlevel)
|
|
stack_trace("[clear]([clear.type])'s turfs_by_zlevel is length [length(clear.turfs_by_zlevel)] but we are being asked to remove turfs from zlevel [area_zlevel] from it.")
|
|
clear.turfs_to_uncontain_by_zlevel[area_zlevel] = list()
|
|
continue
|
|
|
|
// The operation of cutting large lists can be expensive
|
|
// It scales almost directly with the size of the list we're cutting with
|
|
// Because of this, we're gonna stick to cutting 1 entry at a time
|
|
// There's no reason to batch it I promise, this is faster. No overtime too
|
|
var/amount_cut = 0
|
|
var/list/cut_from = clear.turfs_to_uncontain_by_zlevel[area_zlevel]
|
|
for(amount_cut in 1 to length(cut_from))
|
|
clear.turfs_by_zlevel[area_zlevel] -= cut_from[amount_cut]
|
|
if(MC_TICK_CHECK)
|
|
cut_from.Cut(1, amount_cut + 1)
|
|
return
|
|
// avoid double removals if we return early on a subsequent z-level
|
|
clear.turfs_to_uncontain_by_zlevel[area_zlevel] = list()
|
|
|
|
clear.turfs_to_uncontain_by_zlevel = list()
|
|
marked_for_clearing.len--
|
|
|
|
#undef ALLOWED_LOOSE_TURFS
|