Navigation Menu

Skip to content

Commit

Permalink
Fix no sound bug (#5968)
Browse files Browse the repository at this point in the history
  • Loading branch information
Rui authored and nerzhul committed Jun 14, 2017
1 parent ddcd026 commit bbe3dd9
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 8 deletions.
20 changes: 13 additions & 7 deletions src/itemdef.cpp
Expand Up @@ -124,8 +124,8 @@ void ItemDefinition::reset()

void ItemDefinition::serialize(std::ostream &os, u16 protocol_version) const
{

writeU8(os, 3); // version (proto > 20)
u8 version = (protocol_version >= 34) ? 4 : 3;
writeU8(os, version);
writeU8(os, type);
os << serializeString(name);
os << serializeString(description);
Expand Down Expand Up @@ -156,8 +156,11 @@ void ItemDefinition::serialize(std::ostream &os, u16 protocol_version) const
writeF1000(os, sound_place_failed.gain);
os << serializeString(palette_image);
writeU32(os, color.color);
writeF1000(os, sound_place.pitch);
writeF1000(os, sound_place_failed.pitch);

if (version >= 4) {
writeF1000(os, sound_place.pitch);
writeF1000(os, sound_place_failed.pitch);
}
}

void ItemDefinition::deSerialize(std::istream &is)
Expand All @@ -167,7 +170,7 @@ void ItemDefinition::deSerialize(std::istream &is)

// Deserialize
int version = readU8(is);
if(version < 1 || version > 3)
if (version < 1 || version > 4)
throw SerializationError("unsupported ItemDefinition version");
type = (enum ItemType)readU8(is);
name = deSerializeString(is);
Expand Down Expand Up @@ -216,8 +219,11 @@ void ItemDefinition::deSerialize(std::istream &is)
sound_place_failed.gain = readF1000(is);
palette_image = deSerializeString(is);
color.set(readU32(is));
sound_place.pitch = readF1000(is);
sound_place_failed.pitch = readF1000(is);

if (version >= 4) {
sound_place.pitch = readF1000(is);
sound_place_failed.pitch = readF1000(is);
}
} catch(SerializationError &e) {};
}

Expand Down
2 changes: 1 addition & 1 deletion src/sound_openal.cpp
Expand Up @@ -576,7 +576,7 @@ class OpenALSoundManager: public ISoundManager
}
int handle = -1;
if (fade > 0) {
handle = playSoundRaw(buf, loop, 0.0f, 0.0f);
handle = playSoundRaw(buf, loop, 0.0f, pitch);
fadeSound(handle, fade, volume);
} else {
handle = playSoundRaw(buf, loop, volume, pitch);
Expand Down

1 comment on commit bbe3dd9

@paramat
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to cause this bug #5991

Please sign in to comment.