qv4l2: fix a coverity defect
authorHans Verkuil <hans.verkuil@cisco.com>
Fri, 18 Jul 2014 22:21:01 +0000 (00:21 +0200)
committerHans Verkuil <hans.verkuil@cisco.com>
Fri, 18 Jul 2014 22:22:38 +0000 (00:22 +0200)
*** CID 1226677:  Unused pointer value  (UNUSED_VALUE)
/utils/qv4l2/ctrl-tab.cpp: 64 in ApplicationWindow::addWidget(QGridLayout *, QWidget *, QFlags<Qt::AlignmentFlag>)()
58      }
59     }
60
61     void ApplicationWindow::addWidget(QGridLayout *grid, QWidget *w, Qt::Alignment align)
62     {
63      QToolButton *tb;
>>>     CID 1226677:  Unused pointer value  (UNUSED_VALUE)
>>>     Pointer "tb" returned by "qobject_cast(w)" is never used.
64      if (m_col % 2 && !(tb = qobject_cast<QToolButton*>(w)))
65      w->setMinimumWidth(m_minWidth);
66      if (w->sizeHint().width() > m_maxw[m_col])
67      m_maxw[m_col] = w->sizeHint().width();
68      if (w->sizeHint().height() > m_maxh)
69      m_maxh = w->sizeHint().height();

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
utils/qv4l2/ctrl-tab.cpp

index 46a22b5..29f1e27 100644 (file)
@@ -60,8 +60,7 @@ static bool is_valid_type(__u32 type)
 
 void ApplicationWindow::addWidget(QGridLayout *grid, QWidget *w, Qt::Alignment align)
 {
-       QToolButton *tb;
-       if (m_col % 2 && !(tb = qobject_cast<QToolButton*>(w)))
+       if (m_col % 2 && !qobject_cast<QToolButton*>(w))
                w->setMinimumWidth(m_minWidth);
        if (w->sizeHint().width() > m_maxw[m_col])
                m_maxw[m_col] = w->sizeHint().width();