diff options
author | Bardur Arantsson <bardur@scientician.net> | 2015-02-23 09:11:56 +0100 |
---|---|---|
committer | Bardur Arantsson <bardur@scientician.net> | 2015-02-23 09:11:56 +0100 |
commit | 61e8cfc1d553b2bc0cce6b0dd56b03f51f187423 (patch) | |
tree | 4988afbe6ad84804444d0dc3151d300e22d1a80e /src/squelch/automatizer.cc | |
parent | b1f5178bdabafde20eb65df6256b53dc5968b2e2 (diff) |
Clean up warnings about signed/unsigned comparisons
Diffstat (limited to 'src/squelch/automatizer.cc')
-rw-r--r-- | src/squelch/automatizer.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/squelch/automatizer.cc b/src/squelch/automatizer.cc index 0c0f571a..b05a253f 100644 --- a/src/squelch/automatizer.cc +++ b/src/squelch/automatizer.cc @@ -231,9 +231,11 @@ void Automatizer::select_rule(int selected_rule) // Adjust selection to conform to bounds. { + int rules_size = m_rules.size(); // Convert to int to avoid warnings + if (m_selected_rule < 0) { - m_selected_rule = m_rules.size() - 1; + m_selected_rule = rules_size - 1; m_begin = m_selected_rule - hgt + 3; if (m_begin < 0) { @@ -246,7 +248,7 @@ void Automatizer::select_rule(int selected_rule) m_begin = m_selected_rule; } - if (m_selected_rule >= m_rules.size()) + if (m_selected_rule >= rules_size) { m_selected_rule = 0; m_begin = 0; |