• Joined on 2025-01-14
YoyoNow merged pull request SteamWar/SteamWar#178 2025-10-26 15:30:59 +01:00
bausystem/TLS
YoyoNow approved SteamWar/SteamWar#178 2025-10-26 15:30:55 +01:00
bausystem/TLS
YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 15:26:54 +01:00
bausystem/TLS

Noch löschen?

YoyoNow suggested changes for SteamWar/SteamWar#178 2025-10-26 15:26:54 +01:00
bausystem/TLS
YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 15:24:08 +01:00
bausystem/TLS

Die Nachricht braucht es nicht.

YoyoNow suggested changes for SteamWar/SteamWar#178 2025-10-26 15:24:08 +01:00
bausystem/TLS
YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

New Lines

YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

New Lines

YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

@LinkedInstance nutzen

YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

@LinkedInstance nutzen

YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

TLS könnte ein Toggle sein. Warum explizit start und stop?

YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

Selbst sich ne Instanz von einem selber einzufügen ist weird.

YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

Warum nochmal geklammert?

YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

Vllt in eine Method extrahieren und dann durch BauSystem.runTaskLater(<Lambda>, 1); ersetzten?

YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

See: List.isEmpty()

YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

Vllt hier lieber stopListening machen, statt nur einfach zu removen, damit dann auch listening angepasst wird

YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

Könnte ein Set sein und muss nicht ein Map sein. Schaue dir dann auch mal den Rückgabewert von Set.add an

YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

nit: Felder können zusammen geschrieben sein ohne leerzeile

YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

Ein Set wäre hier performanter für das hinzufügen und entfernen, gibt ja jeden Spieler nur 1 mal

YoyoNow commented on pull request SteamWar/SteamWar#178 2025-10-26 14:23:46 +01:00
bausystem/TLS

Muss primedTNT die Region weiterhin als Key haben, wenn es nicht mehr gebraucht wird?