Add implementation notes

This commit is contained in:
Matt Nadareski
2021-10-06 10:24:37 -07:00
parent 879154b0f4
commit a83549824d

View File

@@ -607,13 +607,13 @@ namespace RedBookPlayer.Models.Hardware
/// <summary> /// <summary>
/// Move to the next playable track /// Move to the next playable track
/// </summary> /// </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); public void NextTrack() => SelectTrack(CurrentTrackNumber + 1);
/// <summary> /// <summary>
/// Move to the previous playable track /// Move to the previous playable track
/// </summary> /// </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); public void PreviousTrack() => SelectTrack(CurrentTrackNumber - 1);
/// <summary> /// <summary>
@@ -727,6 +727,7 @@ namespace RedBookPlayer.Models.Hardware
/// Select a disc by number /// Select a disc by number
/// </summary> /// </summary>
/// <param name="discNumber">Disc number to attempt to load</param> /// <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) public void SelectDisc(int discNumber)
{ {
PlayerState wasPlaying = PlayerState; PlayerState wasPlaying = PlayerState;
@@ -827,6 +828,8 @@ namespace RedBookPlayer.Models.Hardware
/// Select a track by number /// Select a track by number
/// </summary> /// </summary>
/// <param name="trackNumber">Track number to attempt to load</param> /// <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) public void SelectTrack(int trackNumber)
{ {
if(_opticalDiscs[CurrentDisc] == null || !_opticalDiscs[CurrentDisc].Initialized) if(_opticalDiscs[CurrentDisc] == null || !_opticalDiscs[CurrentDisc].Initialized)