Add option to disable IPv6

Fixes #50

core.cpp

@@ -63,7 +63,13 @@ Core::~Core()
 
 void Core::start()
 {
-    tox = tox_new(1); // IPv6 enabled, needed for LAN discovery, but can crash some weird routers
+    // IPv6 needed for LAN discovery, but can crash some weird routers. On by default, can be disabled in options.
+    bool enableIPv6 = Settings::getInstance().getEnableIPv6();
+    if (enableIPv6)
+        qDebug() << "Core starting with IPv6 enabled";
+    else
+        qWarning() << "Core starting with IPv6 disabled. LAN discovery may not work properly.";
+    tox = tox_new(enableIPv6);
     if (tox == nullptr)
     {
         qCritical() << "Tox core failed to start";

settings.cpp

@@ -83,6 +83,7 @@ void Settings::load()
     s.beginGroup("General");
         username = s.value("username", "My name").toString();
         statusMessage = s.value("statusMessage", "My status").toString();
+        enableIPv6 = s.value("enableIPv6", true).toBool();
     s.endGroup();
 
     s.beginGroup("Widgets");
@@ -142,6 +143,7 @@ void Settings::save()
     s.beginGroup("General");
         s.setValue("username", username);
         s.setValue("statusMessage", statusMessage);
+        s.setValue("enableIPv6", enableIPv6);
     s.endGroup();
 
     s.beginGroup("Widgets");
@@ -209,6 +211,16 @@ void Settings::setStatusMessage(const QString& newMessage)
     statusMessage = newMessage;
 }
 
+bool Settings::getEnableIPv6() const
+{
+    return enableIPv6;
+}
+
+void Settings::setEnableIPv6(bool newValue)
+{
+    enableIPv6 = newValue;
+}
+
 bool Settings::getEnableLogging() const
 {
     return enableLogging;

settings.h

@@ -49,6 +49,9 @@ public:
     QString getStatusMessage() const;
     void setStatusMessage(const QString& newMessage);
 
+    bool getEnableIPv6() const;
+    void setEnableIPv6(bool newValue);
+
     bool getEnableLogging() const;
     void setEnableLogging(bool newValue);
 
@@ -130,6 +133,8 @@ private:
     QString username;
     QString statusMessage;
 
+    bool enableIPv6;
+
     bool enableLogging;
     bool encryptLogs;
 

widget/form/settingsform.cpp

@@ -1,5 +1,6 @@
 #include "settingsform.h"
 #include "widget/widget.h"
+#include "settings.h"
 #include <QFont>
 
 SettingsForm::SettingsForm()
@@ -19,6 +20,8 @@ SettingsForm::SettingsForm()
     id.setTextInteractionFlags(Qt::TextSelectableByMouse);
 
     videoTest.setText("Test video");
+    enableIPv6.setText("Enable IPv6 (recommended)");
+    enableIPv6.setChecked(Settings::getInstance().getEnableIPv6());
 
     main->setLayout(&layout);
     layout.addWidget(&nameLabel);
@@ -28,12 +31,14 @@ SettingsForm::SettingsForm()
     layout.addWidget(&idLabel);
     layout.addWidget(&id);
     layout.addWidget(&videoTest);
+    layout.addWidget(&enableIPv6);
     layout.addStretch();
 
     head->setLayout(&headLayout);
     headLayout.addWidget(&headLabel);
 
     connect(&videoTest, SIGNAL(clicked()), this, SLOT(onTestVideoClicked()));
+    connect(&enableIPv6, SIGNAL(stateChanged(int)), this, SLOT(onEnableIPv6Updated()));
 }
 
 SettingsForm::~SettingsForm()
@@ -59,3 +64,8 @@ void SettingsForm::onTestVideoClicked()
 {
      Widget::getInstance()->showTestCamview();
 }
+
+void SettingsForm::onEnableIPv6Updated()
+{
+    Settings::getInstance().setEnableIPv6(enableIPv6.isChecked());
+}

widget/form/settingsform.h

@@ -7,6 +7,7 @@
 #include <QString>
 #include <QObject>
 #include <QSpacerItem>
+#include <QCheckBox>
 #include "ui_widget.h"
 #include "widget/selfcamview.h"
 
@@ -24,10 +25,12 @@ public slots:
 
 private slots:
     void onTestVideoClicked();
+    void onEnableIPv6Updated();
 
 private:
     QLabel headLabel, nameLabel, statusTextLabel, idLabel, id;
     QPushButton videoTest;
+    QCheckBox enableIPv6;
     QVBoxLayout layout, headLayout;
     QWidget *main, *head;