Unsigned values are always >= 0. These assertions are redundant, so remove them.
$ grep assert -r ./ |grep ">=" |grep 0
shows the assert(something >= 0). Remove those which assert(unsigned >= 0).
Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
uint64_t LogicSnapshot::get_sample(uint64_t index) const
{
assert(_data);
uint64_t LogicSnapshot::get_sample(uint64_t index) const
{
assert(_data);
- assert(index >= 0 && index < _sample_count);
+ assert(index < _sample_count);
return *(uint64_t*)((uint8_t*)_data + index * _unit_size);
}
return *(uint64_t*)((uint8_t*)_data + index * _unit_size);
}
bool last_sample;
bool fast_forward;
bool last_sample;
bool fast_forward;
assert(end <= get_sample_count());
assert(start <= end);
assert(min_length > 0);
assert(end <= get_sample_count());
assert(start <= end);
assert(min_length > 0);
(level + 1) * MipMapScalePower;
const uint64_t offset =
index >> level_scale_power;
(level + 1) * MipMapScalePower;
const uint64_t offset =
index >> level_scale_power;
// Check if we reached the last block at this
// level, or if there was a change in this block
// Check if we reached the last block at this
// level, or if there was a change in this block
(level + 1) * MipMapScalePower;
const uint64_t offset =
index >> level_scale_power;
(level + 1) * MipMapScalePower;
const uint64_t offset =
index >> level_scale_power;
// Check if we reached the last block at this
// level, or if there was a change in this block
// Check if we reached the last block at this
// level, or if there was a change in this block
} while (tick_period < min_period && unit < countof(ScaleUnits));
const unsigned int prefix = (order - FirstSIPrefixPower) / 3;
} while (tick_period < min_period && unit < countof(ScaleUnits));
const unsigned int prefix = (order - FirstSIPrefixPower) / 3;
assert(prefix < countof(SIPrefixes));
const double multiplier = pow(10.0, - prefix * 3 - FirstSIPrefixPower);
assert(prefix < countof(SIPrefixes));
const double multiplier = pow(10.0, - prefix * 3 - FirstSIPrefixPower);