From 1a09d99a63a3ecab42920c72d02cac07347ec7e0 Mon Sep 17 00:00:00 2001 From: Richard Date: Sun, 13 Sep 2026 14:34:31 +0200 Subject: [PATCH] ServerSettingsFile: Plattformwarnung CA1416 beseitigt Nachtrag zu #560 (68d8d6f). File.SetUnixFileMode stand hinter "mode.HasValue". Das ist inhaltlich richtig, denn mode wird nur ausserhalb von Windows gesetzt. Der Analyzer kann diesen Zusammenhang aber nicht verfolgen und meldete CA1416. Damit war die Zahl der Build-Warnungen von 15 auf 16 gestiegen (#155 fuehrt die 15). Die Plattformpruefung steht jetzt mit in der Bedingung, und es sind wieder 15. Co-Authored-By: Claude Opus 5 --- src/PolyTrader.Core/Configuration/ServerSettingsFile.cs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/PolyTrader.Core/Configuration/ServerSettingsFile.cs b/src/PolyTrader.Core/Configuration/ServerSettingsFile.cs index fd766f8..f6647fb 100644 --- a/src/PolyTrader.Core/Configuration/ServerSettingsFile.cs +++ b/src/PolyTrader.Core/Configuration/ServerSettingsFile.cs @@ -105,7 +105,9 @@ namespace PolyTrader.Core.Configuration fs.Flush(flushToDisk: true); } - if (mode.HasValue) + // Die Plattformpruefung steht mit in der Bedingung, obwohl mode ausserhalb von Windows ohnehin + // null bleibt: Nur so erkennt der Analyzer die Absicherung (CA1416). + if (mode.HasValue && !OperatingSystem.IsWindows()) File.SetUnixFileMode(temp, mode.Value); File.Move(temp, full, overwrite: true);