]> sigrok.org Git - pulseview.git/commitdiff
global: Do not assert >= 0 for unsigned values
authorAlexandru Gagniuc <redacted>
Tue, 8 Jan 2013 20:25:56 +0000 (14:25 -0600)
committerAlexandru Gagniuc <redacted>
Tue, 8 Jan 2013 20:46:16 +0000 (14:46 -0600)
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 <redacted>
pv/data/logicsnapshot.cpp
pv/view/ruler.cpp

index 03e6a54f2e7c097bb2909c9ad1ae1fbe671bbaf5..9b3eedd107ad31f33eab0f0ef45950cb06a0cae0 100644 (file)
@@ -167,7 +167,7 @@ void LogicSnapshot::append_payload_to_mipmap()
 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);
 }
@@ -182,7 +182,6 @@ void LogicSnapshot::get_subsampled_edges(
        bool last_sample;
        bool fast_forward;
 
-       assert(start >= 0);
        assert(end <= get_sample_count());
        assert(start <= end);
        assert(min_length > 0);
@@ -258,7 +257,6 @@ void LogicSnapshot::get_subsampled_edges(
                                        (level + 1) * MipMapScalePower;
                                const uint64_t offset =
                                        index >> level_scale_power;
-                               assert(offset >= 0);
 
                                // Check if we reached the last block at this
                                // level, or if there was a change in this block
@@ -293,7 +291,6 @@ void LogicSnapshot::get_subsampled_edges(
                                        (level + 1) * MipMapScalePower;
                                const uint64_t offset =
                                        index >> level_scale_power;
-                               assert(offset >= 0);
 
                                // Check if we reached the last block at this
                                // level, or if there was a change in this block
index c043aacf2b160d926a557915e2bafe95c397329a..a442901c3e9c76645e44da9a5a9bd5dd18ca8df9 100644 (file)
@@ -79,7 +79,6 @@ void Ruler::paintEvent(QPaintEvent *event)
        } while (tick_period < min_period && unit < countof(ScaleUnits));
 
        const unsigned int prefix = (order - FirstSIPrefixPower) / 3;
-       assert(prefix >= 0);
        assert(prefix < countof(SIPrefixes));
 
        const double multiplier = pow(10.0, - prefix * 3 - FirstSIPrefixPower);