Added TODO: Sword1's MusicHandle::play etc. should be rewritten to take advantage of the enhanced compressed audio stream code

svn-id: r25841

1 files changed, 8 insertions, 0 deletions

diff --git a/engines/sword1/music.cpp b/engines/sword1/music.cpp
index 333c465705..41ca36e306 100644
--- a/engines/sword1/music.cpp
+++ b/engines/sword1/music.cpp
@@ -130,6 +130,14 @@ Audio::AudioStream *MusicHandle::createAudioSource(void) {
 }
 
 bool MusicHandle::play(const char *fileBase, bool loop) {
+/*
+TODO/FIXME: This should be rewritten to make use of the new audio stream factories.
+In particular, it should take advantage of the looping capabilities; and it
+should avoid reading all the data into memory (by not using the old factory functions).
+Essentially, it seems to me as if we could get rid of createAudioSource().
+
+Maybe it could even be change to use AudioStream::openStreamFile.
+*/
 	char fileName[30];
 	stop();
 	_musicMode = MusicNone;