Fix warnings about empty expression statements
This commit is contained in:
parent
0d4d3da41e
commit
a7d359df81
|
@ -139,14 +139,14 @@ template <class StreamType> void VorbisComment::internalParse(StreamType &stream
|
||||||
char sig[8];
|
char sig[8];
|
||||||
bool skipSignature = flags & VorbisCommentFlags::NoSignature;
|
bool skipSignature = flags & VorbisCommentFlags::NoSignature;
|
||||||
if (!skipSignature) {
|
if (!skipSignature) {
|
||||||
CHECK_MAX_SIZE(7);
|
CHECK_MAX_SIZE(7)
|
||||||
stream.read(sig, 7);
|
stream.read(sig, 7);
|
||||||
skipSignature = (BE::toUInt64(sig) & 0xffffffffffffff00u) == 0x03766F7262697300u;
|
skipSignature = (BE::toUInt64(sig) & 0xffffffffffffff00u) == 0x03766F7262697300u;
|
||||||
}
|
}
|
||||||
if (skipSignature) {
|
if (skipSignature) {
|
||||||
// read vendor (length prefixed string)
|
// read vendor (length prefixed string)
|
||||||
{
|
{
|
||||||
CHECK_MAX_SIZE(4);
|
CHECK_MAX_SIZE(4)
|
||||||
stream.read(sig, 4);
|
stream.read(sig, 4);
|
||||||
const auto vendorSize = LE::toUInt32(sig);
|
const auto vendorSize = LE::toUInt32(sig);
|
||||||
if (vendorSize <= maxSize) {
|
if (vendorSize <= maxSize) {
|
||||||
|
@ -161,7 +161,7 @@ template <class StreamType> void VorbisComment::internalParse(StreamType &stream
|
||||||
maxSize -= vendorSize;
|
maxSize -= vendorSize;
|
||||||
}
|
}
|
||||||
// read field count
|
// read field count
|
||||||
CHECK_MAX_SIZE(4);
|
CHECK_MAX_SIZE(4)
|
||||||
stream.read(sig, 4);
|
stream.read(sig, 4);
|
||||||
std::uint32_t fieldCount = LE::toUInt32(sig);
|
std::uint32_t fieldCount = LE::toUInt32(sig);
|
||||||
for (std::uint32_t i = 0; i < fieldCount; ++i) {
|
for (std::uint32_t i = 0; i < fieldCount; ++i) {
|
||||||
|
|
Loading…
Reference in New Issue