Phonon performance slow down

[rspock]

Hi,

i've a problem using the phonon video player. I play a video on loop, but after some hours the application slow down. The memory usage is stable.

I use it in that way:

guiMain::guiMain(QWidget *parent) :
    QWidget(parent),
    ui(new Ui::guiMain)
{
 ...
 
    //video
    video=0;
    QString videoFileName;
    videoFileName.append(conf::Instance()->systemVideoPath);
    videoFileName.append(conf::Instance()->adsVideoFileName);
    QFile videoFile(videoFileName);
    videoRestartTime=new QTimer(this);
    if (videoFile.exists()){
        ui->videoPlayer->show();
        video = new Phonon::MediaSource(videoFileName);
        ui->videoPlayer->play(*video);
        connect(ui->videoPlayer->mediaObject(),SIGNAL(finished()), this,SLOT(videoRestart()));
        pictureEnable=false;
        videoEnable=true;
    }else{
        ui->videoPlayer->hide();
        ui->lblAds->show();
        pictureEnable=true;
        videoEnable=false;
        video=0;
    }
}

guiMain::guiMain(QWidget *parent) :
    QWidget(parent),
    ui(new Ui::guiMain)
{
 ...

    //video
    video=0;
    QString videoFileName;
    videoFileName.append(conf::Instance()->systemVideoPath);
    videoFileName.append(conf::Instance()->adsVideoFileName);
    QFile videoFile(videoFileName);
    videoRestartTime=new QTimer(this);
    if (videoFile.exists()){
        ui->videoPlayer->show();
        video = new Phonon::MediaSource(videoFileName);
        ui->videoPlayer->play(*video);
        connect(ui->videoPlayer->mediaObject(),SIGNAL(finished()), this,SLOT(videoRestart()));
        pictureEnable=false;
        videoEnable=true;
    }else{
        ui->videoPlayer->hide();
        ui->lblAds->show();
        pictureEnable=true;
        videoEnable=false;
        video=0;
    }
}

To copy to clipboard, switch view to plain text mode 

void guiMain::videoRestart(){
    if(videoEnable){
        if (video!=0){
            ui->videoPlayer->stop();
            ui->videoPlayer->load(*video);
            ui->videoPlayer->play();
        }else
            pErrorMng->alarmPush(302,warning,tr("Main Gui"), tr("Ads video not found"));
    }
}

void guiMain::videoRestart(){
    if(videoEnable){
        if (video!=0){
            ui->videoPlayer->stop();
            ui->videoPlayer->load(*video);
            ui->videoPlayer->play();
        }else
            pErrorMng->alarmPush(302,warning,tr("Main Gui"), tr("Ads video not found"));
    }
}

To copy to clipboard, switch view to plain text mode 

the ui->videoPlayer is a Phonon::videoPlayer

What is wrong?

Thanks

[wysota]

When are you deleting the video object?

[rspock]

Never, I create it at the constructor and destroy at the exit.

I pass always the same object to the videoPlayer, do I have to create and destroy the mediaObject and the videoPlayer every time?

The strange thing is that the memory occupancy doesn't increase and if I disable the video(stop and clear) the application return back to the same performance.

[wysota]

Well... first of all you can create that object as a member variable so that you don't have to dereference it everytime you have to pass it to Phonon. Not that it is going to make your code significantly faster but then you didn't show us anything that could help pinpoint the problem so that's the only comment I can give looking at the code you posted.

[rspock]

thanks you for your help.

I post all the class about the video so you can have a look:

guiMain.h

class guiMain : public QWidget
{
    Q_OBJECT
 
public:
    explicit guiMain(QWidget *parent = 0);
    ~guiMain();
...
 
public slots:
 
 
private slots:
 
...
 
    void updateAds();
    void videoRestart();
    void videoAutoRestart();
 
signals:
 
private:
 
...
    //video
    Phonon::MediaSource video;
    QTimer * videoRestartTime;
    void startVideo();
    void stopVideo();
 
};

class guiMain : public QWidget
{
    Q_OBJECT

public:
    explicit guiMain(QWidget *parent = 0);
    ~guiMain();
...

public slots:


private slots:

...

    void updateAds();
    void videoRestart();
    void videoAutoRestart();

signals:

private:

...
    //video
    Phonon::MediaSource video;
    QTimer * videoRestartTime;
    void startVideo();
    void stopVideo();

};

To copy to clipboard, switch view to plain text mode 

guiMain.cpp

guiMain::guiMain(QWidget *parent) :
    QWidget(parent),
    ui(new Ui::guiMain)
{
    ui->setupUi(this);
 
...
    //video
    QString videoFileName;
    videoFileName.append(conf::Instance()->systemVideoPath);
    videoFileName.append(conf::Instance()->adsVideoFileName);
    QFile videoFile(videoFileName);
    videoRestartTime=new QTimer(this);
    if (videoFile.exists()){
        ui->videoPlayer->show();
        video=Phonon::MediaSource (videoFileName);
        ui->videoPlayer->load(video);
        ui->videoPlayer->play();
        connect(ui->videoPlayer->mediaObject(),SIGNAL(aboutToFinish()), this,SLOT(videoAutoRestart()));
        pictureEnable=false;
        videoEnable=true;
    }else{
        ui->videoPlayer->hide();
        ui->lblAds->show();
        pictureEnable=true;
        videoEnable=false;
    }
...
}
 
void guiMain::videoRestart(){
    if(videoEnable && ui->stackedWidget->currentIndex()==nav["pMain"]){
        ui->lblAds->hide();
        ui->videoPlayer->show();
        ui->videoPlayer->load(video);
        ui->videoPlayer->play();
    }
}
 
void guiMain::videoAutoRestart(){
    ui->videoPlayer->mediaObject()->enqueue(video);
    ui->videoPlayer->play();
}
 
void guiMain::stopVideo(){
    ui->videoPlayer->mediaObject()->clearQueue();
    ui->videoPlayer->mediaObject()->clear();
    ui->videoPlayer->stop();
}

guiMain::guiMain(QWidget *parent) :
    QWidget(parent),
    ui(new Ui::guiMain)
{
    ui->setupUi(this);

...
    //video
    QString videoFileName;
    videoFileName.append(conf::Instance()->systemVideoPath);
    videoFileName.append(conf::Instance()->adsVideoFileName);
    QFile videoFile(videoFileName);
    videoRestartTime=new QTimer(this);
    if (videoFile.exists()){
        ui->videoPlayer->show();
        video=Phonon::MediaSource (videoFileName);
        ui->videoPlayer->load(video);
        ui->videoPlayer->play();
        connect(ui->videoPlayer->mediaObject(),SIGNAL(aboutToFinish()), this,SLOT(videoAutoRestart()));
        pictureEnable=false;
        videoEnable=true;
    }else{
        ui->videoPlayer->hide();
        ui->lblAds->show();
        pictureEnable=true;
        videoEnable=false;
    }
...
}

void guiMain::videoRestart(){
    if(videoEnable && ui->stackedWidget->currentIndex()==nav["pMain"]){
        ui->lblAds->hide();
        ui->videoPlayer->show();
        ui->videoPlayer->load(video);
        ui->videoPlayer->play();
    }
}

void guiMain::videoAutoRestart(){
    ui->videoPlayer->mediaObject()->enqueue(video);
    ui->videoPlayer->play();
}

void guiMain::stopVideo(){
    ui->videoPlayer->mediaObject()->clearQueue();
    ui->videoPlayer->mediaObject()->clear();
    ui->videoPlayer->stop();
}

To copy to clipboard, switch view to plain text mode 

The videoRestart is called when I get back to the main screen, the stopVideo is called when i get out of it.
The videoAutoRestart is a slot connected to the aboutToFinish signal