From aa201d92d19cca21bd2456e006a9eb51acafa0e9 Mon Sep 17 00:00:00 2001 From: Rob Mensching Date: Fri, 25 Feb 2022 22:24:52 -0800 Subject: Minor code cleanup --- src/wix/WixToolset.Core/CommandLine/CommandLineArguments.cs | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/src/wix/WixToolset.Core/CommandLine/CommandLineArguments.cs b/src/wix/WixToolset.Core/CommandLine/CommandLineArguments.cs index 40b8b320..0e807be1 100644 --- a/src/wix/WixToolset.Core/CommandLine/CommandLineArguments.cs +++ b/src/wix/WixToolset.Core/CommandLine/CommandLineArguments.cs @@ -41,7 +41,10 @@ namespace WixToolset.Core.CommandLine this.ProcessArgumentsAndParseExtensions(this.OriginalArguments); } - public ICommandLineParser Parse() => new CommandLineParser(this.Messaging, this.Arguments, this.ErrorArgument); + public ICommandLineParser Parse() + { + return new CommandLineParser(this.Messaging, this.Arguments, this.ErrorArgument); + } private void FlattenArgumentsWithResponseFilesIntoOriginalArguments(string[] commandLineArguments) { @@ -49,7 +52,7 @@ namespace WixToolset.Core.CommandLine foreach (var arg in commandLineArguments) { - if (arg != null) + if (!String.IsNullOrEmpty(arg)) { if ('@' == arg[0]) { @@ -202,6 +205,9 @@ namespace WixToolset.Core.CommandLine return arguments; } - private static bool IsSwitchAt(string[] args, int index) => args.Length > index && !String.IsNullOrEmpty(args[index]) && ('/' == args[index][0] || '-' == args[index][0]); + private static bool IsSwitchAt(string[] args, int index) + { + return args.Length > index && !String.IsNullOrEmpty(args[index]) && ('/' == args[index][0] || '-' == args[index][0]); + } } } -- cgit v1.2.3-55-g6feb