aboutsummaryrefslogtreecommitdiff
path: root/sound/mods/module.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'sound/mods/module.cpp')
-rw-r--r--sound/mods/module.cpp5
1 files changed, 5 insertions, 0 deletions
diff --git a/sound/mods/module.cpp b/sound/mods/module.cpp
index 291f95fe3f..d6f1d3eb4d 100644
--- a/sound/mods/module.cpp
+++ b/sound/mods/module.cpp
@@ -33,6 +33,11 @@ bool Module::load(Common::ReadStream &st) {
st.read(songname, 20);
songname[20] = '\0';
+ // FIXME: We define sample to have 32 entries,
+ // yet we only setup 31 of these -- is this on
+ // purpose, or an off-by-one error? This should
+ // be clarified by either adding a comment explaining
+ // this odditiy, or by fixing the off-by-one-bug.
for (int i = 0; i < 31; ++i) {
st.read(sample[i].name, 22);
sample[i].name[22] = '\0';