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

Fix #5006

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;
     }
 }
 

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"));