| author | simon <simon> | 2003-01-01 12:40:02 (UTC) |
|---|---|---|
| committer | simon <simon> | 2003-01-01 12:40:02 (UTC) |
| commit | 25d85fe77aac5763ccc50bd532c2e661f4a64d1b (patch) (side-by-side diff) | |
| tree | 28a4a01f3bf232599db4f6c0d14162d6ac61bbd3 | |
| parent | 31cbc7a5e6cc0a52b58d4de34cc8581a18bc4527 (diff) | |
| download | opie-25d85fe77aac5763ccc50bd532c2e661f4a64d1b.zip opie-25d85fe77aac5763ccc50bd532c2e661f4a64d1b.tar.gz opie-25d85fe77aac5763ccc50bd532c2e661f4a64d1b.tar.bz2 | |
T ORecordList<T>::operator[]( uint i ) {
- if ( i < 0 || (i+1) > m_ids.count() )
+ if ( i >= m_ids.count() )
-> fix warning and simplify :)
| -rw-r--r-- | libopie/pim/orecordlist.h | 2 | ||||
| -rw-r--r-- | libopie2/opiepim/orecordlist.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/libopie/pim/orecordlist.h b/libopie/pim/orecordlist.h index b77a4ab..edcd729 100644 --- a/libopie/pim/orecordlist.h +++ b/libopie/pim/orecordlist.h @@ -260,17 +260,17 @@ typename ORecordList<T>::Iterator ORecordList<T>::end() { return it; } template <class T> uint ORecordList<T>::count()const { return m_ids.count(); } template <class T> T ORecordList<T>::operator[]( uint i ) { - if ( i < 0 || (i+1) > m_ids.count() ) + if ( i >= m_ids.count() ) return T(); /* forward */ return m_acc->find( m_ids[i], m_ids, i ); } template <class T> int ORecordList<T>::uidAt( uint i ) { return m_ids[i]; } diff --git a/libopie2/opiepim/orecordlist.h b/libopie2/opiepim/orecordlist.h index b77a4ab..edcd729 100644 --- a/libopie2/opiepim/orecordlist.h +++ b/libopie2/opiepim/orecordlist.h @@ -260,17 +260,17 @@ typename ORecordList<T>::Iterator ORecordList<T>::end() { return it; } template <class T> uint ORecordList<T>::count()const { return m_ids.count(); } template <class T> T ORecordList<T>::operator[]( uint i ) { - if ( i < 0 || (i+1) > m_ids.count() ) + if ( i >= m_ids.count() ) return T(); /* forward */ return m_acc->find( m_ids[i], m_ids, i ); } template <class T> int ORecordList<T>::uidAt( uint i ) { return m_ids[i]; } |