From c7b582b74421d8a957a752fa73017a458d59fc8c Mon Sep 17 00:00:00 2001 From: adelikat Date: Thu, 12 Dec 2013 23:01:16 +0000 Subject: [PATCH] Actually support control types --- .../movie/MnemonicsGenerator.cs | 32 +++++++++++-------- 1 file changed, 19 insertions(+), 13 deletions(-) diff --git a/BizHawk.Client.Common/movie/MnemonicsGenerator.cs b/BizHawk.Client.Common/movie/MnemonicsGenerator.cs index cc49d34a88..2cc9d3eabb 100644 --- a/BizHawk.Client.Common/movie/MnemonicsGenerator.cs +++ b/BizHawk.Client.Common/movie/MnemonicsGenerator.cs @@ -528,31 +528,37 @@ namespace BizHawk.Client.Common sb.Append('|'); foreach (var mc in collections) { - foreach (var kvp in mc) + if (mc.IsControl) { - if (buttons.ContainsKey(kvp.Key)) + bool anyPressed = false; + foreach (var kvp in mc) { - if (buttons[kvp.Key]) + if (buttons.ContainsKey(kvp.Key) && buttons[kvp.Key]) { sb.Append(kvp.Value); - if (mc.IsControl) - { - break; - } - } - else - { - sb.Append('.'); + anyPressed = true; + break; } } - else + + if (!anyPressed) { sb.Append('.'); } } + else + { + foreach (var kvp in mc) + { + if (buttons.ContainsKey(kvp.Key)) + { + sb.Append(buttons[kvp.Key] ? kvp.Value : '.'); + } + } + } + sb.Append('|'); } - return sb.ToString(); }