chrismanning / melosic Goto Github PK

May need better support for UI extensions in core.

Because of the awkward way output devices were implemented this will be annoying.

Track info source - always once per track

• taglib
• last.fm
• raw file info
• cue file reader

Track info sink - predicated based on various criteria (playback status/position, content, config)

• playback widgets
• last.fm scrobbling
• taglib writer
• cue file writer

Tracks should be AudioSpecs + std::multimap maybe with some convenience functions for common fields like title, artist, album, etc.

Add support for reading CUE sheets as playlists with separate tracks possibly pointing to a different part of a single file.

Support files containing more than one track, eg. album images with external/embedded CUE sheets.

• Add the ability to create and modify tag metadata in various formats on various file types.
• Create basic GUI dialog for editing "standard" fields eg. Artist, Album, Title, TrackNumber, etc.
• Expand on tag format specifiers to allow tagging from filenames, other fields, etc.

Create an MP3 plugin for decoding. Maybe using libmad

Default layout should be functional and informative.

Player needs less state.

It should do nothing other than take bytes from a pcm data source and push into an output sink. It should not handle playlists, track skipping, seeking etc.

State machine should distinguish between states and transitions better.

Current config can move to a new class representing the streaming pcm data source which can handle tracks & playlists.

A new Player can be created every time it's requested to start playing from a new source. This Player should be initialised with a prepared output device and a pcm data source. Player can yield control when either the data source ends (Stopped) or the audio specs change (Error?).

States

• Playing
• Paused
• Stopped
• Error

class Player {
    std::unique_ptr<std::basic_istream<char>> pcm_data_stream;
    std::unique_ptr<output::device> output_device;
    Player(std::unique_ptr<std::basic_istream<char>>, std::unique_ptr<output::device>);
};

Implement provider-y stuff for config