Standard C++ callbacks handling inside Qt application

[mranger90]

I'm not sure how this would work with a lambda. You might try to create a static member function and pass a pointer to that function. With a static function, you are going to need a pointer to mainWindow, since you wont get "this". A lot of libraries that use callbacks allow you to pass a pointer to "this" as a void * argument for just this purpose.
i.e.
MainWindow.h

static void onNewSystem();

MainWindow.cpp

mavsdk.subscribe_on_new_system(&MainWindow::onNewSystem);

MainWindow::onNewSystem()
{
   // assume that a pointer to MainWindow is set somewhere
   mainWindow->setWindowTitle("Connected.");

[SimonSchroeder]

@Dragoon said in Standard C++ callbacks handling inside Qt application:

AFAIK this llibrary uses different threads to manage such callback.

This might be one more problem. You need to call setWindowTitle() within the main GUI thread. In addition to previous fixes suggested, why don't you just write to the console if your lambda function gets called? Because it might actually be called but not do what you expect.

The easiest way to have something done in the GUI thread is for your callback to just trigger a signal which is connected to a slot that does the actual work (i.e. call setWindowTitle()).

[Dragoon]

@SimonSchroeder I'll try this approach...

[Dragoon]

@SimonSchroeder tried and failed.

 missionServer->subscribe_incoming_mission(
   [&](mavsdk::MissionRawServer::Result result, mavsdk::MissionRawServer::MissionPlan plan)
   {
      emit incomingMission(result, plan);
   }
);

void MainWindow::onIncomingMission(MissionRawServer::Result result, MissionRawServer::MissionPlan plan)
{   
       Q_UNUSED(result);
       Q_UNUSED(plan);
        missions++;
}

Debugging it I can see callback being triggered but incominMission() never being fired.

[Dragoon]

@JonB the third party library could not be the main issue in this. The problem is the management of C++ 17 standard callbacks.
And, BTW, even:

missions++;

(being mission an int variable) is never executed.

[jsulm]

@Dragoon Did you make sure you connected incomingMission successfully to onIncomingMission? Did you check the lifetime of the involved objects? If the lambda is called then the signal is for sure emitted. Most probably you did not connect signal/slot.

[Dragoon]

@mranger90
defining 'slots' as static member seem to work.
The problem now is to access MainWindow instance...

[jsulm]

@Dragoon said in Standard C++ callbacks handling inside Qt application:

defining 'slots' as static member seem to work

Why static?!
Can you show how and what you're actually connecting?

[Dragoon]

@jsulm said in Standard C++ callbacks handling inside Qt application:

Most probably you did not connect signal/slot.

I'm not SO newbie... ;-)

MainWindow::MainWindow(QWidget *parent)
    : QMainWindow(parent)
    , ui(new Ui::MainWindow)
    , configuration(Mavsdk::Configuration::UsageType::Autopilot)
    , eventTimer(new QTimer(this))

{
    ui->setupUi(this);

    connect(this, &MainWindow::newSystem, this, &MainWindow::onNewSystem);
    connect(this, &MainWindow::incomingMission, this, &MainWindow::onIncomingMission);
    connect(eventTimer, &QTimer::timeout, this, &MainWindow::onUpdateEvent);

    initMAVSDK();
    initMAVConnection();
}

void MainWindow::initMAVSDK()
{
    configuration.set_system_id(DEF_ID);

    mavsdk.set_configuration(configuration);
}

void MainWindow::initMAVConnection()
{
    ConnectionResult conn_result = mavsdk.add_any_connection(DEF_URL.toStdString());  

    if(conn_result == mavsdk::ConnectionResult::Success)
    {
        mavsdk.subscribe_on_new_system(
            [&](){
                emit newSystem();
            }
        );

        eventTimer->start(DEF_UPDATE_TIMEOUT);
    }
}

[Dragoon]

it seems that I found the real issue...
I need to put these lines:

Q_DECLARE_METATYPE(MissionRawServer::Result)
Q_DECLARE_METATYPE(MissionRawServer::MissionPlan)

in my class header

and these:

qRegisterMetaType<MissionRawServer::Result>();
qRegisterMetaType<MissionRawServer::MissionPlan>();

prior connect calls.

Probably since parameters on callback where not QObject derivative they needed to be registered to be connect proof.

I don't know it this's the correct register procedure and explanation of this issue but now ti seem to be fixed.
Could someone confirm?

[jsulm]

@Dragoon said in Standard C++ callbacks handling inside Qt application:

I need to put these lines

Ah, yes you're right. If you use your own custom data types as parameters for signals/slots you have to register them.

[Kent-Dorfman]

@Dragoon so should this be marked as solved?

[Dragoon]

@Kent-Dorfman I do.