]> git.sesse.net Git - kdenlive/blobdiff - src/monitor.cpp
Make monitor context menu consistent with "Monitor" menu
[kdenlive] / src / monitor.cpp
index 5ac13d2a7586d7ae9bc7f5b0bf3149ef8e9713c7..2b1102784169660d7d96afa777e549d5cbd5000d 100644 (file)
 #include <QMenu>
 #include <QToolButton>
 #include <QToolBar>
+#include <QDesktopWidget>
 
 #include <KDebug>
 #include <KLocale>
+#include <KFileDialog>
+#include <KApplication>
 
 #include "gentime.h"
 #include "monitor.h"
+#include "renderer.h"
+#include "monitormanager.h"
+#include "smallruler.h"
+#include "docclipbase.h"
 
 Monitor::Monitor(QString name, MonitorManager *manager, QWidget *parent)
-        : QWidget(parent), render(NULL), m_monitorManager(manager), m_name(name), m_isActive(false) {
+        : QWidget(parent), render(NULL), m_monitorManager(manager), m_name(name), m_isActive(false), m_currentClip(NULL), m_dragStarted(false) {
     ui.setupUi(this);
     m_scale = 1;
     m_ruler = new SmallRuler();
     QVBoxLayout *layout = new QVBoxLayout;
+    layout->setContentsMargins(0, 0, 0, 0);
     layout->addWidget(m_ruler);
     ui.ruler_frame->setLayout(layout);
-
+    setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Expanding);
+    setMinimumHeight(200);
     QToolBar *toolbar = new QToolBar(name, this);
     QVBoxLayout *layout2 = new QVBoxLayout;
+    layout2->setContentsMargins(0, 0, 0, 0);
 
     m_playIcon = KIcon("media-playback-start");
     m_pauseIcon = KIcon("media-playback-pause");
 
-    QAction *m_rewAction = toolbar->addAction(KIcon("media-seek-backward"), i18n("Rewind"));
-    connect(m_rewAction, SIGNAL(triggered()), this, SLOT(slotRewind()));
-    QAction *m_rew1Action = toolbar->addAction(KIcon("media-skip-backward"), i18n("Rewind 1 frame"));
-    connect(m_rew1Action, SIGNAL(triggered()), this, SLOT(slotRewindOneFrame()));
+    QAction *zoneStart = toolbar->addAction(KIcon("kdenlive-zone-start"), i18n("Set zone start"), this, SLOT(slotSetZoneStart()));
+    QAction *zoneEnd = toolbar->addAction(KIcon("kdenlive-zone-end"), i18n("Set zone end"), this, SLOT(slotSetZoneEnd()));
 
-    QToolButton *playButton = new QToolButton(toolbar);
-    QMenu *playMenu = new QMenu(this);
-    playButton->setMenu(playMenu);
-    playButton->setPopupMode(QToolButton::MenuButtonPopup);
-    toolbar->addWidget(playButton);
+    toolbar->addAction(KIcon("media-seek-backward"), i18n("Rewind"), this, SLOT(slotRewind()));
+    toolbar->addAction(KIcon("media-skip-backward"), i18n("Rewind 1 frame"), this, SLOT(slotRewindOneFrame()));
 
-    m_playAction = playMenu->addAction(m_playIcon, i18n("Play"));
+    QToolButton *playButton = new QToolButton(toolbar);
+    m_playMenu = new QMenu(i18n("Play..."), this);
+    m_playAction = m_playMenu->addAction(m_playIcon, i18n("Play"));
     m_playAction->setCheckable(true);
     connect(m_playAction, SIGNAL(triggered()), this, SLOT(slotPlay()));
-    QAction *m_playSectionAction = playMenu->addAction(m_playIcon, i18n("Play Section"));
-    connect(m_playSectionAction, SIGNAL(triggered()), this, SLOT(slotPlay()));
-    QAction *m_loopSectionAction = playMenu->addAction(m_playIcon, i18n("Loop Section"));
-    connect(m_loopSectionAction, SIGNAL(triggered()), this, SLOT(slotPlay()));
+    m_playMenu->addAction(m_playIcon, i18n("Play Section"), this, SLOT(slotPlayZone()));
+    m_playMenu->addAction(m_playIcon, i18n("Loop Section"), this, SLOT(slotLoopZone()));
+
+    playButton->setMenu(m_playMenu);
+    playButton->setPopupMode(QToolButton::MenuButtonPopup);
+    toolbar->addWidget(playButton);
 
-    QAction *m_fwd1Action = toolbar->addAction(KIcon("media-skip-forward"), i18n("Forward 1 frame"));
-    connect(m_fwd1Action, SIGNAL(triggered()), this, SLOT(slotForwardOneFrame()));
-    QAction *m_fwdAction = toolbar->addAction(KIcon("media-seek-forward"), i18n("Forward"));
-    connect(m_fwdAction, SIGNAL(triggered()), this, SLOT(slotForward()));
+    toolbar->addAction(KIcon("media-skip-forward"), i18n("Forward 1 frame"), this, SLOT(slotForwardOneFrame()));
+    toolbar->addAction(KIcon("media-seek-forward"), i18n("Forward"), this, SLOT(slotForward()));
 
     playButton->setDefaultAction(m_playAction);
 
+    QToolButton *configButton = new QToolButton(toolbar);
+    m_configMenu = new QMenu(i18n("Misc..."), this);
+    configButton->setIcon(KIcon("system-run"));
+    configButton->setMenu(m_configMenu);
+    configButton->setPopupMode(QToolButton::QToolButton::InstantPopup);
+    toolbar->addWidget(configButton);
+
+    QWidget *spacer = new QWidget(this);
+    spacer->setSizePolicy(QSizePolicy::MinimumExpanding, QSizePolicy::MinimumExpanding);
+    toolbar->addWidget(spacer);
     m_timePos = new KRestrictedLine(this);
+    m_timePos->setSizePolicy(QSizePolicy::Maximum, QSizePolicy::MinimumExpanding);
     m_timePos->setInputMask("99:99:99:99");
     toolbar->addWidget(m_timePos);
 
+    connect(m_timePos, SIGNAL(editingFinished()), this, SLOT(slotSeek()));
+
     layout2->addWidget(toolbar);
     ui.button_frame->setLayout(layout2);
+    const int toolHeight = toolbar->height();
+    ui.button_frame->setMinimumHeight(toolHeight);
 
     //m_ruler->setPixelPerMark(3);
 
-
     QVBoxLayout *rendererBox = new QVBoxLayout(ui.video_frame);
+    rendererBox->setContentsMargins(0, 0, 0, 0);
     m_monitorRefresh = new MonitorRefresh(ui.video_frame);
     rendererBox->addWidget(m_monitorRefresh);
-    m_monitorRefresh->setAttribute(Qt::WA_PaintOnScreen);
     render = new Render(m_name, (int) m_monitorRefresh->winId(), -1, this);
     m_monitorRefresh->setRenderer(render);
 
+
+
+    connect(m_ruler, SIGNAL(seekRenderer(int)), this, SLOT(slotSeek(int)));
+    connect(m_ruler, SIGNAL(zoneChanged(QPoint)), this, SIGNAL(zoneUpdated(QPoint)));
     connect(render, SIGNAL(durationChanged(int)), this, SLOT(adjustRulerSize(int)));
     connect(render, SIGNAL(rendererPosition(int)), this, SLOT(seekCursor(int)));
     connect(render, SIGNAL(rendererStopped(int)), this, SLOT(rendererStopped(int)));
+
+    m_configMenu->addSeparator();
+    m_configMenu->addAction(KIcon("transform-scale"), i18n("Resize (100%)"), this, SLOT(slotSetSizeOneToOne()));
+    m_configMenu->addAction(KIcon("transform-scale"), i18n("Resize (50%)"), this, SLOT(slotSetSizeOneToTwo()));
+    //render->createVideoXWindow(ui.video_frame->winId(), -1);
+    m_length = 0;
+
     if (name != "clip") {
         connect(render, SIGNAL(rendererPosition(int)), this, SIGNAL(renderPosition(int)));
         connect(render, SIGNAL(durationChanged(int)), this, SIGNAL(durationChanged(int)));
     }
-    //render->createVideoXWindow(ui.video_frame->winId(), -1);
-    int width = m_ruler->width();
-    m_ruler->setLength(width);
-    m_ruler->setMaximum(width);
-    m_length = 0;
-
+    m_monitorRefresh->show();
     kDebug() << "/////// BUILDING MONITOR, ID: " << ui.video_frame->winId();
 }
 
@@ -108,45 +135,265 @@ QString Monitor::name() const {
     return m_name;
 }
 
+void Monitor::setupMenu(QMenu *goMenu) {
+    m_contextMenu = new QMenu(this);
+    m_contextMenu->addMenu(m_playMenu);
+    m_contextMenu->addMenu(goMenu);
+
+    //TODO: add save zone to timeline monitor when fixed
+    if (m_name == "clip") m_contextMenu->addAction(KIcon("document-save"), i18n("Save zone"), this, SLOT(slotSaveZone()));
+
+    QAction *extractFrame = m_configMenu->addAction(KIcon("document-new"), i18n("Extract frame"), this, SLOT(slotExtractCurrentFrame()));
+    m_contextMenu->addAction(extractFrame);
+
+    if (m_name != "clip") {
+        QAction *splitView = m_contextMenu->addAction(KIcon("document-new"), i18n("Split view"), render, SLOT(slotSplitView(bool)));
+        splitView->setCheckable(true);
+        m_configMenu->addAction(splitView);
+    } else {
+        QAction *setThumbFrame = m_contextMenu->addAction(KIcon("document-new"), i18n("Set current image as thumbnail"), this, SLOT(slotSetThumbFrame()));
+        m_configMenu->addAction(setThumbFrame);
+    }
+
+}
+
+void Monitor::slotSetSizeOneToOne() {
+    QRect r = QApplication::desktop()->screenGeometry();
+    const int maxWidth = r.width() - 20;
+    const int maxHeight = r.height() - 20;
+    int width = render->renderWidth();
+    int height = render->renderHeight();
+    kDebug() << "// render info: " << width << "x" << height;
+    while (width >= maxWidth || height >= maxHeight) {
+        width = width * 0.8;
+        height = height * 0.8;
+    }
+    kDebug() << "// MONITOR; set SIZE: " << width << ", " << height;
+    ui.video_frame->setFixedSize(width, height);
+    updateGeometry();
+    adjustSize();
+    //ui.video_frame->setMinimumSize(0, 0);
+    emit adjustMonitorSize();
+}
+
+void Monitor::slotSetSizeOneToTwo() {
+    QRect r = QApplication::desktop()->screenGeometry();
+    const int maxWidth = r.width() - 20;
+    const int maxHeight = r.height() - 20;
+    int width = render->renderWidth() / 2;
+    int height = render->renderHeight() / 2;
+    kDebug() << "// render info: " << width << "x" << height;
+    while (width >= maxWidth || height >= maxHeight) {
+        width = width * 0.8;
+        height = height * 0.8;
+    }
+    kDebug() << "// MONITOR; set SIZE: " << width << ", " << height;
+    ui.video_frame->setFixedSize(width, height);
+    updateGeometry();
+    adjustSize();
+    //ui.video_frame->setMinimumSize(0, 0);
+    emit adjustMonitorSize();
+}
+
+void Monitor::resetSize() {
+    ui.video_frame->setMinimumSize(0, 0);
+}
+
+void Monitor::slotZoneMoved(int start, int end) {
+    m_ruler->setZone(start, end);
+}
+
+void Monitor::slotSetZoneStart() {
+    m_ruler->setZone(m_position, -1);
+    emit zoneUpdated(m_ruler->zone());
+}
+
+void Monitor::slotSetZoneEnd() {
+    m_ruler->setZone(-1, m_position);
+    emit zoneUpdated(m_ruler->zone());
+}
+
 // virtual
 void Monitor::mousePressEvent(QMouseEvent * event) {
-    slotPlay();
+    if (event->button() != Qt::RightButton) {
+        if (ui.video_frame->underMouse()) {
+            m_dragStarted = true;
+            m_DragStartPosition = event->pos();
+        }
+    } else m_contextMenu->popup(event->globalPos());
+}
+
+// virtual
+void Monitor::mouseReleaseEvent(QMouseEvent * event) {
+    if (m_dragStarted) {
+        if (ui.video_frame->underMouse()) {
+            slotPlay();
+        } else QWidget::mouseReleaseEvent(event);
+        m_dragStarted = false;
+    }
 }
 
+
+// virtual
+void Monitor::mouseMoveEvent(QMouseEvent *event) {
+    kDebug() << "// DRAG STARTED, MOUSE MOVED: ";
+    if (!m_dragStarted || m_currentClip == NULL) return;
+
+    if ((event->pos() - m_DragStartPosition).manhattanLength()
+            < QApplication::startDragDistance())
+        return;
+
+    {
+        QDrag *drag = new QDrag(this);
+        QMimeData *mimeData = new QMimeData;
+
+        QStringList list;
+        list.append(m_currentClip->getId());
+        QPoint p = m_ruler->zone();
+        list.append(QString::number(p.x()));
+        list.append(QString::number(p.y()));
+        QByteArray data;
+        data.append(list.join(";").toUtf8());
+        mimeData->setData("kdenlive/clip", data);
+        drag->setMimeData(mimeData);
+        QPixmap pix = m_currentClip->thumbnail();
+        kDebug() << "/ / / /CLIP DRAGGED PIXMAP: " << pix.width() << "x" << pix.height();
+        drag->setPixmap(pix);
+        drag->setHotSpot(QPoint(0, 50));
+        drag->start(Qt::MoveAction);
+
+        //Qt::DropAction dropAction;
+        //dropAction = drag->start(Qt::CopyAction | Qt::MoveAction);
+
+        //Qt::DropAction dropAction = drag->exec();
+
+    }
+    //event->accept();
+}
+
+/*void Monitor::dragMoveEvent(QDragMoveEvent * event) {
+    event->setDropAction(Qt::IgnoreAction);
+    event->setDropAction(Qt::MoveAction);
+    if (event->mimeData()->hasText()) {
+        event->acceptProposedAction();
+    }
+}
+
+Qt::DropActions Monitor::supportedDropActions() const {
+    // returns what actions are supported when dropping
+    return Qt::MoveAction;
+}*/
+
+QStringList Monitor::mimeTypes() const {
+    QStringList qstrList;
+    // list of accepted mime types for drop
+    qstrList.append("kdenlive/clip");
+    return qstrList;
+}
+
+
 // virtual
 void Monitor::wheelEvent(QWheelEvent * event) {
-    if (event->delta() > 0) slotForwardOneFrame();
-    else slotRewindOneFrame();
+    if (event->modifiers() == Qt::ControlModifier) {
+        int delta = m_monitorManager->timecode().fps();
+        if (event->delta() < 0) delta = 0 - delta;
+        slotSeek(m_position + delta);
+    } else {
+        if (event->delta() > 0) slotForwardOneFrame();
+        else slotRewindOneFrame();
+    }
+}
+
+void Monitor::slotSetThumbFrame() {
+    if (m_currentClip == NULL) {
+        return;
+    }
+    m_currentClip->setClipThumbFrame((uint) m_position);
+    emit refreshClipThumbnail(m_currentClip->getId());
+}
+
+void Monitor::slotExtractCurrentFrame() {
+    QPixmap frame = render->extractFrame(m_position);
+    QString outputFile = KFileDialog::getSaveFileName(KUrl(), "image/png");
+    if (!outputFile.isEmpty()) frame.save(outputFile);
+}
+
+bool Monitor::isActive() const {
+    return m_isActive;
 }
 
 void Monitor::activateMonitor() {
     if (!m_isActive) m_monitorManager->activateMonitor(m_name);
 }
 
+void Monitor::slotSeek() {
+    const int frames = m_monitorManager->timecode().getFrameCount(m_timePos->text(), m_monitorManager->timecode().fps());
+    slotSeek(frames);
+}
+
 void Monitor::slotSeek(int pos) {
     if (!m_isActive) m_monitorManager->activateMonitor(m_name);
     if (render == NULL) return;
-    int realPos = ((double) pos) / m_scale;
-    render->seekToFrame(realPos);
-    m_position = realPos;
+    render->seekToFrame(pos);
+    m_position = pos;
+    emit renderPosition(m_position);
+    m_timePos->setText(m_monitorManager->timecode().getTimecodeFromFrames(m_position));
+}
+
+void Monitor::slotStart() {
+    if (!m_isActive) m_monitorManager->activateMonitor(m_name);
+    render->play(0);
+    m_position = 0;
+    render->seekToFrame(m_position);
+    emit renderPosition(m_position);
+    m_timePos->setText(m_monitorManager->timecode().getTimecodeFromFrames(m_position));
+}
+
+void Monitor::slotEnd() {
+    if (!m_isActive) m_monitorManager->activateMonitor(m_name);
+    render->play(0);
+    m_position = render->getLength();
+    render->seekToFrame(m_position);
+    emit renderPosition(m_position);
+    m_timePos->setText(m_monitorManager->timecode().getTimecodeFromFrames(m_position));
+}
+
+void Monitor::slotZoneStart() {
+    if (!m_isActive) m_monitorManager->activateMonitor(m_name);
+    render->play(0);
+    m_position = m_ruler->zone().x();
+    render->seekToFrame(m_position);
     emit renderPosition(m_position);
     m_timePos->setText(m_monitorManager->timecode().getTimecodeFromFrames(m_position));
 }
 
-void Monitor::slotRewind() {
+void Monitor::slotZoneEnd() {
     if (!m_isActive) m_monitorManager->activateMonitor(m_name);
-    double speed = render->playSpeed();
-    if (speed >= 0) render->play(-2);
-    else render->play(speed * 2);
+    render->play(0);
+    m_position = m_ruler->zone().y();
+    render->seekToFrame(m_position);
+    emit renderPosition(m_position);
+    m_timePos->setText(m_monitorManager->timecode().getTimecodeFromFrames(m_position));
+}
+
+void Monitor::slotRewind(double speed) {
+    if (!m_isActive) m_monitorManager->activateMonitor(m_name);
+    if (speed == 0) {
+        double currentspeed = render->playSpeed();
+        if (currentspeed >= 0) render->play(-2);
+        else render->play(currentspeed * 2);
+    } else render->play(speed);
     m_playAction->setChecked(true);
     m_playAction->setIcon(m_pauseIcon);
 }
 
-void Monitor::slotForward() {
+void Monitor::slotForward(double speed) {
     if (!m_isActive) m_monitorManager->activateMonitor(m_name);
-    double speed = render->playSpeed();
-    if (speed <= 1) render->play(2);
-    else render->play(speed * 2);
+    if (speed == 0) {
+        double currentspeed = render->playSpeed();
+        if (currentspeed <= 1) render->play(2);
+        else render->play(currentspeed * 2);
+    } else render->play(speed);
     m_playAction->setChecked(true);
     m_playAction->setIcon(m_pauseIcon);
 }
@@ -173,16 +420,14 @@ void Monitor::slotForwardOneFrame() {
 
 void Monitor::seekCursor(int pos) {
     if (!m_isActive) m_monitorManager->activateMonitor(m_name);
-    int rulerPos = (int)(pos * m_scale);
     m_position = pos;
     m_timePos->setText(m_monitorManager->timecode().getTimecodeFromFrames(pos));
-    //kDebug() << "seek: " << pos << ", scale: " << m_scale;
-    m_ruler->slotNewValue(rulerPos);
+    m_ruler->slotNewValue(pos);
 }
 
 void Monitor::rendererStopped(int pos) {
-    int rulerPos = (int)(pos * m_scale);
-    m_ruler->slotNewValue(rulerPos);
+    //int rulerPos = (int)(pos * m_scale);
+    m_ruler->slotNewValue(pos);
     m_position = pos;
     m_timePos->setText(m_monitorManager->timecode().getTimecodeFromFrames(pos));
     m_playAction->setChecked(false);
@@ -194,42 +439,45 @@ void Monitor::initMonitor() {
 }
 
 // virtual
-void Monitor::resizeEvent(QResizeEvent * event) {
+/*void Monitor::resizeEvent(QResizeEvent * event) {
     QWidget::resizeEvent(event);
     adjustRulerSize(-1);
     if (render && m_isActive) render->doRefresh();
     //
-}
+}*/
 
 void Monitor::adjustRulerSize(int length) {
-    int width = m_ruler->width();
-    m_ruler->setLength(width);
     if (length > 0) m_length = length;
-    m_scale = (double) width / m_length;
-    if (m_scale == 0) m_scale = 1;
-    kDebug() << "RULER WIDT: " << width << ", RENDER LENGT: " << m_length << ", SCALE: " << m_scale;
-    m_ruler->setPixelPerMark(m_scale);
-    m_ruler->setMaximum(width);
-    //m_ruler->setLength(length);
+    m_ruler->adjustScale(m_length);
 }
 
 void Monitor::stop() {
     m_isActive = false;
     if (render) render->stop();
+    //kDebug()<<"/// MONITOR RENDER STOP";
 }
 
 void Monitor::start() {
     m_isActive = true;
     if (render) render->start();
+    //kDebug()<<"/// MONITOR RENDER START";
 }
 
 void Monitor::refreshMonitor(bool visible) {
     if (visible && render) {
         if (!m_isActive) m_monitorManager->activateMonitor(m_name);
-        render->askForRefresh();
+        render->doRefresh(); //askForRefresh();
     }
 }
 
+void Monitor::pause() {
+    if (render == NULL) return;
+    if (!m_isActive) m_monitorManager->activateMonitor(m_name);
+    render->pause();
+    //m_playAction->setChecked(true);
+    //m_playAction->setIcon(m_pauseIcon);
+}
+
 void Monitor::slotPlay() {
     if (render == NULL) return;
     if (!m_isActive) m_monitorManager->activateMonitor(m_name);
@@ -238,17 +486,36 @@ void Monitor::slotPlay() {
     m_playAction->setIcon(m_pauseIcon);
 }
 
-void Monitor::slotSetXml(const QDomElement &e) {
+void Monitor::slotPlayZone() {
     if (render == NULL) return;
     if (!m_isActive) m_monitorManager->activateMonitor(m_name);
-    QDomDocument doc;
-    QDomElement westley = doc.createElement("westley");
-    doc.appendChild(westley);
-    westley.appendChild(e);
-    render->setSceneList(doc, 0);
-    m_ruler->slotNewValue(0);
-    m_timePos->setText("00:00:00:00");
-    m_position = 0;
+    QPoint p = m_ruler->zone();
+    render->playZone(GenTime(p.x(), render->fps()), GenTime(p.y(), render->fps()));
+    m_playAction->setChecked(true);
+    m_playAction->setIcon(m_pauseIcon);
+}
+
+void Monitor::slotLoopZone() {
+    if (render == NULL) return;
+    if (!m_isActive) m_monitorManager->activateMonitor(m_name);
+    QPoint p = m_ruler->zone();
+    render->loopZone(GenTime(p.x(), render->fps()), GenTime(p.y(), render->fps()));
+    m_playAction->setChecked(true);
+    m_playAction->setIcon(m_pauseIcon);
+}
+
+void Monitor::slotSetXml(DocClipBase *clip, const int position) {
+    if (render == NULL) return;
+    if (!m_isActive) m_monitorManager->activateMonitor(m_name);
+    if (!clip) return;
+    if (clip != m_currentClip && clip->producer() != NULL) {
+        m_currentClip = clip;
+        render->setProducer(clip->producer(), position);
+        //m_ruler->slotNewValue(0);
+        //adjustRulerSize(clip->producer()->get_playtime());
+        //m_timePos->setText("00:00:00:00");
+        m_position = position;
+    } else if (position != -1) render->seek(GenTime(position, render->fps()));
 }
 
 void Monitor::slotOpenFile(const QString &file) {
@@ -264,18 +531,27 @@ void Monitor::slotOpenFile(const QString &file) {
     render->setSceneList(doc, 0);
 }
 
-void Monitor::resetProfile(QString prof) {
+void Monitor::slotSaveZone() {
     if (render == NULL) return;
-    render->resetProfile(prof);
+    emit saveZone(render, m_ruler->zone());
+
+    //render->setSceneList(doc, 0);
 }
 
-void Monitor::saveSceneList(QString path, QDomElement e) {
+
+void Monitor::resetProfile() {
     if (render == NULL) return;
-    render->saveSceneList(path, e);
+    render->resetProfile();
 }
 
-MonitorRefresh::MonitorRefresh(QWidget* parent): QWidget(parent), m_renderer(NULL) {
+void Monitor::saveSceneList(QString path, QDomElement info) {
+    if (render == NULL) return;
+    render->saveSceneList(path, info);
+}
 
+MonitorRefresh::MonitorRefresh(QWidget* parent): QWidget(parent), m_renderer(NULL) {
+    setAttribute(Qt::WA_PaintOnScreen);
+    setAttribute(Qt::WA_OpaquePaintEvent); //setAttribute(Qt::WA_NoSystemBackground);
 }
 
 void MonitorRefresh::setRenderer(Render* render) {
@@ -283,7 +559,7 @@ void MonitorRefresh::setRenderer(Render* render) {
 }
 
 void MonitorRefresh::paintEvent(QPaintEvent * event) {
-    if (m_renderer != NULL) m_renderer->doRefresh();
+    if (m_renderer) m_renderer->doRefresh();
 }
 
 #include "monitor.moc"