fix(build): Remove unused vars, add default case for switch

Fix #5006
qTox · Mar 16, 2018 · bc18990 · bc18990
1 parent 1afed5f
commit bc18990
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/core/core.cpp b/src/core/core.cpp
@@ -713,6 +713,10 @@ bool parseConferenceSendMessageError(TOX_ERR_CONFERENCE_SEND_MESSAGE error)
     case TOX_ERR_CONFERENCE_SEND_MESSAGE_TOO_LONG:
         qCritical() << "Message too long";
         return false;
+
+    default:
+        qCritical() << "Unknown TOX_ERR_CONFERENCE_SEND_MESSAGE error";
+        return false;
     }
 }
 

diff --git a/src/widget/friendwidget.cpp b/src/widget/friendwidget.cpp
@@ -139,10 +139,6 @@ void FriendWidget::onContextMenuCalled(QContextMenuEvent* event)
 
     const auto& s = Settings::getInstance();
     const auto circleId = s.getFriendCircleID(frnd->getPublicKey());
-    auto circleWidget = CircleWidget::getFromID(circleId);
-    auto w = circleWidget ? circleWidget : static_cast<QWidget*>(this);
-    auto friendList = qobject_cast<FriendListWidget*>(w->parentWidget());
-
     auto circleMenu = menu.addMenu(tr("Move to circle...",
                                         "Menu to move a friend into a different circle"));