Remove more unused render code (#141)

Author: highbyte

src/apps/Highbyte.DotNet6502.App.SadConsole/SadConsoleHostApp.cs

@@ -3,7 +3,7 @@
 using Highbyte.DotNet6502.Impl.NAudio;
 using Highbyte.DotNet6502.Impl.NAudio.NAudioOpenALProvider;
 using Highbyte.DotNet6502.Impl.SadConsole;
-using Highbyte.DotNet6502.Impl.SadConsole.Commodore64.Video;
+using Highbyte.DotNet6502.Impl.SadConsole.Commodore64.Render;
 using Highbyte.DotNet6502.Systems;
 using Highbyte.DotNet6502.Systems.Commodore64;
 using Highbyte.DotNet6502.Systems.Logging.InMem;
@@ -128,7 +128,6 @@ public void Run()
             .SetStartingScreen(CreateMainSadConsoleScreen)
             .IsStartingScreenFocused(false) // Let the object focused in the create method remain.
             .AddFrameUpdateEvent(UpdateSadConsole);
-        //.AddFrameRenderEvent(RenderSadConsole); // Old rendering pipeline
 
         Settings.WindowTitle = "Highbyte.DotNet6502 emulator + SadConsole (with NAudio)";
         Settings.ResizeMode = Settings.WindowResizeOptions.None;

src/apps/Highbyte.DotNet6502.App.SilkNetNative/SilkNetHostApp.cs

@@ -128,7 +128,6 @@ public void Run()
         _window.Load += OnLoad;
         _window.Closing += OnClosing;
         _window.Update += OnUpdate;
-        //_window.Render += OnRender; // old rendering pipeline
         _window.Resize += OnResize;
 
         _window.Run();

…64SadConsoleRenderTargetCustomization.cs …64SadConsoleRenderTargetCustomization.cssrc/libraries/Highbyte.DotNet6502.Impl.SadConsole/Commodore64/Video/C64SadConsoleRenderTargetCustomization.cs renamed to src/libraries/Highbyte.DotNet6502.Impl.SadConsole/Commodore64/Render/C64SadConsoleRenderTargetCustomization.cs src/libraries/Highbyte.DotNet6502.Impl.SadConsole/Commodore64/Video/C64SadConsoleRenderTargetCustomization.cs renamed to src/libraries/Highbyte.DotNet6502.Impl.SadConsole/Commodore64/Render/C64SadConsoleRenderTargetCustomization.cs

@@ -1,6 +1,6 @@
 using Highbyte.DotNet6502.Systems.Commodore64;
 
-namespace Highbyte.DotNet6502.Impl.SadConsole.Commodore64.Video;
+namespace Highbyte.DotNet6502.Impl.SadConsole.Commodore64.Render;
 public class C64SadConsoleRenderTargetCustomization
 {
     private readonly C64 _c64;

src/libraries/Highbyte.DotNet6502.Impl.SadConsole/Commodore64/Video/C64SadConsoleColors.cs

@@ -28,7 +28,7 @@ namespace Highbyte.DotNet6502.Impl.Skia.Commodore64.Render.Legacy.v2;
 /// - Fine scroll per frame.
 /// - Sprites (Standard, MultiColor)
 ///   
-/// Compared to C64SkiaRenderer:
+/// Compared to C64LegacyRenderTarget:
 /// - A bit slower, but supports Bitmap graphics and still able to run in a browser with Blazor WebAssembly.
 /// </summary>
 [DisplayName("Skia legacy v2")]

src/libraries/Highbyte.DotNet6502.Impl.SadConsole/Commodore64/Video/C64SadConsoleRenderer.cs

@@ -182,7 +182,6 @@ public List<Type> GetAvailableSystemRenderProviderTypes()
         return available;
     }
 
-
     public async Task SelectSystem(string systemName)
     {
         if (EmulatorState != EmulatorState.Uninitialized)
@@ -387,7 +386,6 @@ public void RunEmulatorOneFrame()
 
     public virtual void OnAfterRunEmulatorOneFrame(ExecEvaluatorTriggerResult execEvaluatorTriggerResult) { }
 
-    // New rendering pipeline
     private void InitRendererForSystem()
     {
         // Skip rendering initialization if no render config has been provided (e.g., in unit tests)

src/libraries/Highbyte.DotNet6502.Impl.Skia/Commodore64/Render/Legacy/v2/C64LegacyRenderTarget2.cs

@@ -28,16 +28,12 @@ public class SystemRunner
 
     public SystemRunner(ISystem system, IInputHandler inputHandler, IAudioHandler audioHandler)
     {
-        // if (system != renderer.System)
-        //     throw new DotNet6502Exception("Renderer must be for the same system as the SystemRunner.");
         if (system != inputHandler.System)
             throw new DotNet6502Exception("InputHandler must be for the same system as the SystemRunner.");
         if (system != audioHandler.System)
             throw new DotNet6502Exception("AudioHandler must be for the same system as the SystemRunner.");
 
         _system = system;
-        //_renderer = renderer; // Old renderer pipeline
-        //_renderer = renderer ?? new NullRenderer(system);
         _inputHandler = inputHandler;
         _audioHandler = audioHandler;
     }
@@ -84,17 +80,8 @@ public ExecEvaluatorTriggerResult RunEmulatorOneFrame()
         return execEvaluatorTriggerResult;
     }
 
-    /// <summary>
-    /// Called by host app that runs the emulator, once per frame tied to the host app rendering frequency.
-    /// </summary>
-    public void Draw()
-    {
-        //_renderer.DrawFrame();
-    }
-
     public void Cleanup()
     {
-        //_renderer?.Cleanup(); // Old renderer pipeline
         _audioHandler.Cleanup();
         _inputHandler.Cleanup();
     }