From 07f7ea581ca985d54a1284d9384a8e223532fb37 Mon Sep 17 00:00:00 2001 From: Anna Clemens Date: Wed, 7 Apr 2021 19:11:54 -0400 Subject: [PATCH] refactor: fix wait parse to be invariant --- Macrology/Configuration.cs | 6 +++--- Macrology/ILRepack.targets | 2 +- Macrology/MacroHandler.cs | 11 ++++++----- Macrology/Macrology.csproj | 2 +- 4 files changed, 11 insertions(+), 10 deletions(-) diff --git a/Macrology/Configuration.cs b/Macrology/Configuration.cs index bd853dc..9187ef5 100644 --- a/Macrology/Configuration.cs +++ b/Macrology/Configuration.cs @@ -15,7 +15,7 @@ namespace Macrology { [JsonProperty] [JsonConverter(typeof(NodeConverter))] - public List Nodes { get; private set; } = new List(); + public List Nodes { get; private set; } = new(); public int MaxLength { get; set; } = 10_000; @@ -86,7 +86,7 @@ namespace Macrology { public class Folder : INode { public Guid Id { get; set; } public string Name { get; set; } - public List Children { get; private set; } = new List(); + public List Children { get; private set; } = new(); public Folder(string name, List? children = null) { this.Id = Guid.NewGuid(); @@ -111,7 +111,7 @@ namespace Macrology { public Guid Id { get; set; } public string Name { get; set; } public string Contents { get; set; } - public List Children => new List(); + public List Children => new(); public Macro(string name, string contents) { this.Id = Guid.NewGuid(); diff --git a/Macrology/ILRepack.targets b/Macrology/ILRepack.targets index 272ed9a..2e6bbd4 100644 --- a/Macrology/ILRepack.targets +++ b/Macrology/ILRepack.targets @@ -11,7 +11,7 @@ InputAssemblies="@(InputAssemblies)" TargetKind="Dll" TargetPlatformVersion="v4" - LibraryPath="$(OutputPath)" + LibraryPath="$(OutputPath);$(AppData)\XIVLauncher\addon\Hooks\dev" OutputFile="$(OutputPath)\$(AssemblyName).dll"/> diff --git a/Macrology/MacroHandler.cs b/Macrology/MacroHandler.cs index aea647f..0b3ca31 100644 --- a/Macrology/MacroHandler.cs +++ b/Macrology/MacroHandler.cs @@ -1,6 +1,7 @@ using Dalamud.Game.Internal; using System; using System.Collections.Concurrent; +using System.Globalization; using System.Linq; using System.Text.RegularExpressions; using System.Threading.Channels; @@ -9,7 +10,7 @@ using System.Threading.Tasks; namespace Macrology { public class MacroHandler { private bool _ready; - private static readonly Regex Wait = new Regex(@"", RegexOptions.Compiled | RegexOptions.IgnoreCase); + private static readonly Regex Wait = new(@"", RegexOptions.Compiled | RegexOptions.IgnoreCase); private static readonly string[] FastCommands = { "/ac", @@ -20,9 +21,9 @@ namespace Macrology { private Macrology Plugin { get; } private readonly Channel _commands = Channel.CreateUnbounded(); - public ConcurrentDictionary Running { get; } = new ConcurrentDictionary(); - private readonly ConcurrentDictionary _cancelled = new ConcurrentDictionary(); - private readonly ConcurrentDictionary _paused = new ConcurrentDictionary(); + public ConcurrentDictionary Running { get; } = new(); + private readonly ConcurrentDictionary _cancelled = new(); + private readonly ConcurrentDictionary _paused = new(); public MacroHandler(Macrology plugin) { this.Plugin = plugin ?? throw new ArgumentNullException(nameof(plugin), "Macrology cannot be null"); @@ -155,7 +156,7 @@ namespace Macrology { var match = matches[matches.Count - 1]; var waitTime = match.Groups[1].Captures[0].Value; - if (!double.TryParse(waitTime, out var seconds)) { + if (!double.TryParse(waitTime, NumberStyles.Number, CultureInfo.InvariantCulture, out var seconds)) { return null; } diff --git a/Macrology/Macrology.csproj b/Macrology/Macrology.csproj index 968b4e2..35ffb79 100755 --- a/Macrology/Macrology.csproj +++ b/Macrology/Macrology.csproj @@ -8,7 +8,7 @@ true - + $(AppData)\XIVLauncher\addon\Hooks\dev\Dalamud.dll False