Details
-
Bug
-
Resolution: Done
-
P3: Somewhat important
-
5.2.0 RC1
-
None
-
494882adadb33ffb7a7a9b2fa9356362df87d3bf
Description
Static analyser cppcheck just said
1.
[data/qdaytimeduration.cpp:170] -> [data/qdaytimeduration.cpp:170]: (style) Same expression on both sides of '||'.
Source code is
if(m_seconds || m_seconds)
maybe
if(m_seconds || m_mseconds)
was intended.
2.
[data/qdaytimeduration.cpp:148] -> [data/qdaytimeduration.cpp:148]: (style) Same expression on both sides of '&&'.
if(!m_hours && !m_minutes && !m_seconds && !m_seconds)
Duplicate.