From 36538641284bdec1a8104edc6fb6eec5a0441a88 Mon Sep 17 00:00:00 2001 From: Natalia Portillo Date: Sun, 13 Nov 2022 21:14:16 +0000 Subject: [PATCH] Invert 'if' statements to reduce nesting. --- MediaTypeFromDevice/FromAta.cs | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/MediaTypeFromDevice/FromAta.cs b/MediaTypeFromDevice/FromAta.cs index aa067e2..c00acdd 100644 --- a/MediaTypeFromDevice/FromAta.cs +++ b/MediaTypeFromDevice/FromAta.cs @@ -57,16 +57,15 @@ public static partial class MediaTypeFromDevice return pcmcia ? MediaType.PCCardTypeI : MediaType.GENERIC_HDD; } - if(manufacturer.ToLowerInvariant() == "syquest" && - model.ToLowerInvariant() == "sparq" && - blocks == 1961069) - { - AaruConsole.DebugWriteLine("Media detection", - "Drive manufacturer is SyQuest, media has 1961069 blocks of 512 bytes, setting media type to SparQ."); + if(manufacturer.ToLowerInvariant() != "syquest" || + model.ToLowerInvariant() != "sparq" || + blocks != 1961069) + return MediaType.Unknown; - return MediaType.SparQ; - } + AaruConsole.DebugWriteLine("Media detection", + "Drive manufacturer is SyQuest, media has 1961069 blocks of 512 bytes, setting media type to SparQ."); + + return MediaType.SparQ; - return MediaType.Unknown; } } \ No newline at end of file