App crashes when using QCoreApplication in library

[jamalabo]

@kshegunov I don't understand where should I put the QCoreApplication? if the GUI is running on QApplication

In GUI

int main(int argc, char *argv[])
{
    QApplication a(argc, argv);
    MainWindow w;
    w.set_arguments(argc, argv);
    w.show();
    return a.exec();
}

question: should I use the same QxxxApplication for the GUI and the LIB? and if so I get the error QEventLoop: Cannot be used without QApplication and If I try to pass the app/MainWindow as the parent it's throw the error QObject: Cannot create children for a parent that is in a different thread

[kshegunov]

No, there's only one QApplication object, the one in main()! Also I really see no reason for you getting the thread errors, are you using threads?

[jamalabo]

@kshegunov no but I'm calling the code on a slot in MainWindow (on button click), (I don't know if this runs on different thread than MainWindow)

[kshegunov]

@jamalabo said in App crashes when using QCoreApplication in library:

I don't know if this runs on different thread than MainWindow

It doesn't, then I imagine you have static QObjects that are initialized before they're allowed to, a.k.a. "the QApplication object must always be the first one created and the last destroyed". So, as said, create your application object, then pass control to your library to do whatever it does, and when that's done continue on to start the event loop with exec().

Your code looks more or less okay, assuming you've removed all QApplication creation from the MainWindow and/or related classes.

[jamalabo]

@kshegunov it seems like it's working but not properly

void MY_LIB::DownloadableContent::loadHeaders() {
    printf("load-headers\n");
    QNetworkAccessManager manager;
    printf("manager\n");
    connect(&manager, &QNetworkAccessManager::finished, this, &DownloadableContent::http_headers_loaded);
    manager.head(this->gen_request());
}

void MY_LIB::DownloadableContent::http_headers_loaded(QNetworkReply *reply) {
    printf("headers-loaded");
    emit headersLoaded();
}

after running this code I only get the output

load-headers
manager

it seems like slots are not called and it's an event loop issue.
should I pass the QParent (app) param to QNetworkAcessManager?

[JKSH]

@jamalabo said in App crashes when using QCoreApplication in library:

I don't understand where should I put the QCoreApplication? if the GUI is running on QApplication

• QApplication inherits QCoreApplication.
• You can only have one instance of Q___Application in your program.

So, you cannot create both a QApplication and a QCoreApplication in the same program.

question: should I use the same QxxxApplication for the GUI and the LIB?

Yes. Definitely. They must share the same QxxxApplication instance. (And since your code uses widgets, then you must use QApplication)

[kshegunov]

@jamalabo said in App crashes when using QCoreApplication in library:

it seems like slots are not called and it's an event loop issue.

Well, that doesn't surprise me, your QNetworkAccessManager object dies as you leave the function. So either you make it a member to your DownloadableContent class, or create it on the heap (parented properly). And if you decide on the latter, please don't do it every time you pass through that method, just the first time.

should I pass the QParent (app) param to QNetworkAcessManager?

Not necessarily, although usually you give parents to QObject instances. Since DownloadableContent already derives from QObject, what's wrong with giving the QNetworkAcessManager object a parent of the object which "owns"/"contains" it (a.k.a. this in this case)?

[jamalabo]

@kshegunov
now it crashes.

QNetworkAccessManager *NETWORK_COMBINER_LIB::DownloadableContent::get_manager() {
    return (!this->manager ? (this->manager = new QNetworkAccessManager()) : this->manager);
}

void NETWORK_COMBINER_LIB::DownloadableContent::loadHeaders(QObject *parent) {
    printf("load-headers\n");
    QNetworkAccessManager *pManager = this->get_manager();
    printf("pManager\n");
    connect(pManager, &QNetworkAccessManager::finished, this, &DownloadableContent::http_headers_loaded);
    pManager->head(this->gen_request());
}

[JKSH]

@jamalabo said in App crashes when using QCoreApplication in library:

now it crashes.

Look at your debugger's stack trace to see why it crashed.

!this->manager ?

Did you initialize the pointer to nullptr?

[jamalabo]

@JKSH yes it's initialized to nullptr but still does nothing (the request is not sent)

[JKSH]

@jamalabo said in App crashes when using QCoreApplication in library:

yes it's initialized to nullptr

OK, so it sounds like initializing nullptr stopped the crash.

but still does nothing (the request is not sent)

Time to start debugging.

• What does your QNetworkRequest look like?
• Does the QNetworkAccessManager emit any other signals after you called head()? https://doc.qt.io/qt-5/qnetworkaccessmanager.html#signals
• Does the QNetworkReply emit any signals after you called head()? (The QNetworkReply object is returned by QNetworkAccessManager::head()) https://doc.qt.io/qt-5/qnetworkreply.html#signals