From 144a07f0882b79183b17d477b900beffe67f3c73 Mon Sep 17 00:00:00 2001 From: thebrick448 Date: Tue, 7 Jan 2014 05:53:57 +0000 Subject: [PATCH] Fixed loading files on TI83. --- BizHawk.Emulation.Cores/Calculator/TI83.cs | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/BizHawk.Emulation.Cores/Calculator/TI83.cs b/BizHawk.Emulation.Cores/Calculator/TI83.cs index 12deb24cee..58fc6cb42c 100644 --- a/BizHawk.Emulation.Cores/Calculator/TI83.cs +++ b/BizHawk.Emulation.Cores/Calculator/TI83.cs @@ -673,6 +673,11 @@ namespace BizHawk.Emulation.Cores.Calculators public class Link { + //Emulates TI linking software. + //See http://www.ticalc.org/archives/files/fileinfo/294/29418.html for documentation + + //Note: Each hardware read/write to the link port calls tthe update method. + readonly TI83 Parent; private FileStream CurrentFile; @@ -994,7 +999,7 @@ namespace BizHawk.Emulation.Cores.Calculators CurrentData.Enqueue(0x00); CurrentStatus = Status.PrepareReceive; - NextStep = Finalize; + NextStep = FinalizeFile; Parent.LinkActive = true; } @@ -1021,20 +1026,20 @@ namespace BizHawk.Emulation.Cores.Calculators CurrentData.Enqueue(0x00); CurrentStatus = Status.PrepareReceive; - NextStep = Finalize; + NextStep = FinalizeFile; Parent.LinkActive = true; } //adelikat: This isn't used (yet?) and causes a warning. Did you really mean to override finalize? if so it should be protected virtual, else rename this - /* - private void Finalize() + //Envian: I didn't notice there was a naming conflict. Fixed it since this method is necessary for the file upload proccess. + private void FinalizeFile() { + //Resets the link software, and checks to see if there is an additional file to send. CurrentData.Clear(); Parent.LinkActive = false; NextStep = null; SendNextFile(); } - */ } public object GetSettings() { return null; }