Access and inheritance

[mrjj]

@tomy
Hi
The line it shows when you click on 11 ? line 1339

[tomy]

@mrjj, Hi,

Here you are:

[mrjj]

Hi
it was not what i as i had hoped, it being inside some of the user code.
I wonder if you have a double free.
Did you change anything in main.cpp?

[tomy]

@mrjj Hi,

Here is main.cpp:

#include "myspreadsheet.h"
#include <QApplication>
#include <QSplashScreen>

int main(int argc, char *argv[])
{
    QApplication app(argc, argv);

    QSplashScreen* splash = new QSplashScreen;
    splash->setPixmap(QPixmap(":/images/splash.jpg"));
    splash->show();

    Qt::Alignment topRight = Qt::AlignRight | Qt::AlignTop;
    splash->showMessage(QObject::tr("Setting up the main window..."), topRight, Qt::white);


    MySpreadsheet* mySP = new MySpreadsheet;
    if(argc > 1)
        mySP -> loadFile(argv[1]);

    mySP -> show();
    splash -> finish(mySP);
    delete splash;

    return app.exec();
}

I added those splash stuff and now apparently it's fine!

[mrjj]

Hi
Does look fine.
Try set breakpoint in main.cpp and press f10 to signle step over it and
see if we can find the reason for the crash.
Since its first on close, it might be inside MySpreadsheet indirectly.

[tomy]

@mrjj
Hi, It's fine now. :) Thanks so much. :)

[mrjj]

@tomy
Hi
What was it ? Just to learn also.

[tomy]

@mrjj Hi,

The faulty one didn't include splash stuff and was this way I think:

#include "myspreadsheet.h"
#include <QApplication>

int main(int argc, char *argv[])
{
    QApplication app(argc, argv);
    MySpreadsheet* mySP = new MySpreadsheet;
    mySP -> show();

    return app.exec();
}

Then I added the splash stuff. Now it "apparently" is fine, but I'm still not sure not do I know of the exact reason for that red message either. :(

[SGaist]

You are not deleting mySP. The memory will be cleaned up at the end of the application but this doesn't mean that the destructor will be called. So if you have anything there that does something it's not guaranteed to be called.

You should rather keep it on the stack or delete it after returning from app.exec();