From a83549824db75236e5f31beeec9bf153f2761e51 Mon Sep 17 00:00:00 2001
From: Matt Nadareski <mnadareski@outlook.com>
Date: Wed, 6 Oct 2021 10:24:37 -0700
Subject: [PATCH] Add implementation notes

---
 RedBookPlayer.Models/Hardware/Player.cs | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/RedBookPlayer.Models/Hardware/Player.cs b/RedBookPlayer.Models/Hardware/Player.cs
index 712c2aa..d2da495 100644
--- a/RedBookPlayer.Models/Hardware/Player.cs
+++ b/RedBookPlayer.Models/Hardware/Player.cs
@@ -607,13 +607,13 @@ namespace RedBookPlayer.Models.Hardware
         /// <summary>
         /// Move to the next playable track
         /// </summary>
-        /// <remarks>This should follow the track playback order</remarks>
+        /// <remarks>TODO: This should follow the track playback order</remarks>
         public void NextTrack() => SelectTrack(CurrentTrackNumber + 1);
 
         /// <summary>
         /// Move to the previous playable track
         /// </summary>
-        /// <remarks>This should follow the track playback order</remarks>
+        /// <remarks>TODO: This should follow the track playback order</remarks>
         public void PreviousTrack() => SelectTrack(CurrentTrackNumber - 1);
 
         /// <summary>
@@ -727,6 +727,7 @@ namespace RedBookPlayer.Models.Hardware
         /// Select a disc by number
         /// </summary>
         /// <param name="discNumber">Disc number to attempt to load</param>
+        /// <remarks>TODO: This needs to reset the pointer in the track playback order</remarks>
         public void SelectDisc(int discNumber)
         {
             PlayerState wasPlaying = PlayerState;
@@ -827,6 +828,8 @@ namespace RedBookPlayer.Models.Hardware
         /// Select a track by number
         /// </summary>
         /// <param name="trackNumber">Track number to attempt to load</param>
+        /// <remarks>TODO: This needs to reset the pointer in the track playback order</remarks>
+        /// <remarks>TODO: There needs to be a SelectRelativeTrack variant that follows order and then invokes this</remarks>
         public void SelectTrack(int trackNumber)
         {
             if(_opticalDiscs[CurrentDisc] == null || !_opticalDiscs[CurrentDisc].Initialized)