X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=src%2Frenderer.cpp;h=3826e933e24edd87714baa2dae2264ef0df8d656;hb=61a66ae9f58000c64d3ca82aef1978eee1bdc871;hp=00e691b677522dc9c6408fc4ac6e2cf902ca60a4;hpb=d92f43b96847b44c8efbb06cd889612ab7fee0f7;p=kdenlive diff --git a/src/renderer.cpp b/src/renderer.cpp index 00e691b6..3826e933 100644 --- a/src/renderer.cpp +++ b/src/renderer.cpp @@ -95,13 +95,11 @@ static void consumer_gl_frame_show(mlt_consumer, Render * self, mlt_frame frame_ { // detect if the producer has finished playing. Is there a better way to do it? Mlt::Frame frame(frame_ptr); - self->showFrame(frame); - if (frame.get_double("_speed") == 0.0) { - self->emitConsumerStopped(); - } else if (frame.get_double("_speed") < 0.0 && mlt_frame_get_position(frame_ptr) <= 0) { - self->pause(); - self->emitConsumerStopped(); + if (frame.get_double("_speed") < 0.0 && mlt_frame_get_position(frame_ptr) <= 0) { + self->pause(); + self->emitConsumerStopped(); } + self->showFrame(frame); } Render::Render(Kdenlive::MONITORID rendererName, int winid, QString profile, QWidget *parent) : @@ -128,9 +126,10 @@ Render::Render(Kdenlive::MONITORID rendererName, int winid, QString profile, QWi m_mltConsumer->connect(*m_mltProducer); m_mltProducer->set_speed(0.0); m_refreshTimer.setSingleShot(true); - m_refreshTimer.setInterval(70); + m_refreshTimer.setInterval(100); connect(&m_refreshTimer, SIGNAL(timeout()), this, SLOT(refresh())); connect(this, SIGNAL(multiStreamFound(const QString &,QList,QList,stringMap)), this, SLOT(slotMultiStreamProducerFound(const QString &,QList,QList,stringMap))); + connect(this, SIGNAL(checkSeeking()), this, SLOT(slotCheckSeeking())); } Render::~Render() @@ -193,15 +192,23 @@ void Render::buildConsumer(const QString &profileName) delete m_blackClip; m_blackClip = NULL; - //TODO: uncomment following line when everything is clean - // uncommented Feb 2011 --Granjow - if (m_mltProfile) delete m_mltProfile; m_activeProfile = profileName; - char *tmp = qstrdup(m_activeProfile.toUtf8().constData()); - setenv("MLT_PROFILE", tmp, 1); - m_mltProfile = new Mlt::Profile(tmp); + if (m_mltProfile) { + Mlt::Profile tmpProfile(m_activeProfile.toUtf8().constData()); + m_mltProfile->set_colorspace(tmpProfile.colorspace()); + m_mltProfile->set_frame_rate(tmpProfile.frame_rate_num(), tmpProfile.frame_rate_den()); + m_mltProfile->set_height(tmpProfile.height()); + m_mltProfile->set_width(tmpProfile.width()); + m_mltProfile->set_progressive(tmpProfile.progressive()); + m_mltProfile->set_sample_aspect(tmpProfile.sample_aspect_num(), tmpProfile.sample_aspect_den()); + m_mltProfile->get_profile()->display_aspect_num = tmpProfile.display_aspect_num(); + m_mltProfile->get_profile()->display_aspect_den = tmpProfile.display_aspect_den(); + } + else { + m_mltProfile = new Mlt::Profile(m_activeProfile.toUtf8().constData()); + } + setenv("MLT_PROFILE", m_activeProfile.toUtf8().constData(), 1); m_mltProfile->set_explicit(true); - delete[] tmp; m_blackClip = new Mlt::Producer(*m_mltProfile, "colour", "black"); m_blackClip->set("id", "black"); @@ -215,16 +222,16 @@ void Render::buildConsumer(const QString &profileName) if (device >= 0) { if (BMInterface::isSupportedProfile(device, profileProperties)) { QString decklink = "decklink:" + QString::number(KdenliveSettings::blackmagic_output_device()); - tmp = qstrdup(decklink.toUtf8().constData()); - m_mltConsumer = new Mlt::Consumer(*m_mltProfile, tmp); - delete[] tmp; + if (!m_mltConsumer) { + m_mltConsumer = new Mlt::Consumer(*m_mltProfile, decklink.toUtf8().constData()); + m_showFrameEvent = m_mltConsumer->listen("consumer-frame-show", this, (mlt_listener) consumer_frame_show); + mlt_log_set_callback(kdenlive_callback); + } if (m_mltConsumer->is_valid()) { m_externalConsumer = true; - m_showFrameEvent = m_mltConsumer->listen("consumer-frame-show", this, (mlt_listener) consumer_frame_show); m_mltConsumer->set("terminate_on_pause", 0); m_mltConsumer->set("deinterlace_method", "onefield"); m_mltConsumer->set("real_time", KdenliveSettings::mltthreads()); - mlt_log_set_callback(kdenlive_callback); } if (m_mltConsumer && m_mltConsumer->is_valid()) return; } else KMessageBox::informationList(qApp->activeWindow(), i18n("Your project's profile %1 is not compatible with the blackmagic output card. Please see supported profiles below. Switching to normal video display.", m_mltProfile->description()), BMInterface::supportedModes(KdenliveSettings::blackmagic_output_device())); @@ -246,15 +253,19 @@ void Render::buildConsumer(const QString &profileName) //m_mltConsumer->set("fullscreen", 1); if (m_winid == 0) { // OpenGL monitor - m_mltConsumer = new Mlt::Consumer(*m_mltProfile, "sdl_audio"); + if (!m_mltConsumer) { + m_mltConsumer = new Mlt::Consumer(*m_mltProfile, "sdl_audio"); + m_showFrameEvent = m_mltConsumer->listen("consumer-frame-show", this, (mlt_listener) consumer_gl_frame_show); + } m_mltConsumer->set("preview_off", 1); m_mltConsumer->set("preview_format", mlt_image_rgb24a); - m_showFrameEvent = m_mltConsumer->listen("consumer-frame-show", this, (mlt_listener) consumer_gl_frame_show); } else { - m_mltConsumer = new Mlt::Consumer(*m_mltProfile, "sdl_preview"); - m_showFrameEvent = m_mltConsumer->listen("consumer-frame-show", this, (mlt_listener) consumer_frame_show); - m_pauseEvent = m_mltConsumer->listen("consumer-sdl-paused", this, (mlt_listener) consumer_paused); - m_mltConsumer->set("window_id", m_winid); + if (!m_mltConsumer) { + m_mltConsumer = new Mlt::Consumer(*m_mltProfile, "sdl_preview"); + m_showFrameEvent = m_mltConsumer->listen("consumer-frame-show", this, (mlt_listener) consumer_frame_show); + m_pauseEvent = m_mltConsumer->listen("consumer-sdl-paused", this, (mlt_listener) consumer_paused); + } + m_mltConsumer->set("window_id", m_winid); } m_mltConsumer->set("resize", 1); m_mltConsumer->set("terminate_on_pause", 1); @@ -328,12 +339,6 @@ int Render::resetProfile(const QString &profileName, bool dropSceneList) if (m_isSplitView) slotSplitView(false); if (!m_mltConsumer->is_stopped()) m_mltConsumer->stop(); m_mltConsumer->purge(); - if (m_showFrameEvent) delete m_showFrameEvent; - m_showFrameEvent = NULL; - if (m_pauseEvent) delete m_pauseEvent; - m_pauseEvent = NULL; - delete m_mltConsumer; - m_mltConsumer = NULL; } QString scene; if (!dropSceneList) scene = sceneList(); @@ -391,7 +396,7 @@ void Render::seek(int time) if (requestedSeekPosition == SEEK_INACTIVE) { requestedSeekPosition = time; m_mltProducer->seek(time); - m_mltConsumer->purge(); + //m_mltConsumer->purge(); if (m_mltProducer->get_speed() == 0) { refresh(); } @@ -532,7 +537,8 @@ void Render::slotSplitView(bool doit) transition->set("mlt_service", "composite"); transition->set("a_track", 0); transition->set("b_track", i); - transition->set("distort", 1); + transition->set("distort", 0); + transition->set("aligned", 0); transition->set("internal_added", "200"); QString geometry; switch (screen) { @@ -670,7 +676,6 @@ void Render::processFileProperties() KUrl url(path); Mlt::Producer *producer = NULL; CLIPTYPE type = (CLIPTYPE)info.xml.attribute("type").toInt(); - if (type == COLOR) { producer = new Mlt::Producer(*m_mltProfile, 0, ("colour:" + info.xml.attribute("colour")).toUtf8().constData()); } else if (type == TEXT) { @@ -678,15 +683,23 @@ void Render::processFileProperties() if (producer && producer->is_valid() && info.xml.hasAttribute("xmldata")) producer->set("xmldata", info.xml.attribute("xmldata").toUtf8().constData()); } else if (url.isEmpty()) { + //WARNING: when is this case used? Not sure it is working.. JBM/ QDomDocument doc; QDomElement mlt = doc.createElement("mlt"); QDomElement play = doc.createElement("playlist"); + play.setAttribute("id", "playlist0"); doc.appendChild(mlt); mlt.appendChild(play); play.appendChild(doc.importNode(info.xml, true)); + QDomElement tractor = doc.createElement("tractor"); + tractor.setAttribute("id", "tractor0"); + QDomElement track = doc.createElement("track"); + track.setAttribute("producer", "playlist0"); + tractor.appendChild(track); + mlt.appendChild(tractor); producer = new Mlt::Producer(*m_mltProfile, "xml-string", doc.toString().toUtf8().constData()); } else { - producer = new Mlt::Producer(*m_mltProfile, path.toUtf8().constData()); + producer = new Mlt::Producer(*m_mltProfile, path.toUtf8().constData()); } if (producer == NULL || producer->is_blank() || !producer->is_valid()) { @@ -771,12 +784,12 @@ void Render::processFileProperties() if (type == COLOR || type == TEXT || type == IMAGE || type == SLIDESHOW) { int length; if (info.xml.hasAttribute("length")) { - if (clipOut > 0) duration = clipOut + 1; length = info.xml.attribute("length").toInt(); clipOut = length - 1; } else length = info.xml.attribute("out").toInt() - info.xml.attribute("in").toInt(); producer->set("length", length); + duration = length; } if (clipOut > 0) producer->set_in_and_out(info.xml.attribute("in").toInt(), clipOut); @@ -812,7 +825,6 @@ void Render::processFileProperties() char property[200]; if (frameNumber > 0) producer->seek(frameNumber); - duration = duration > 0 ? duration : producer->get_playtime(); filePropertyMap["duration"] = QString::number(duration); //kDebug() << "/////// PRODUCER: " << url.path() << " IS: " << producer->get_playtime(); @@ -865,13 +877,17 @@ void Render::processFileProperties() int vindex = -1; const QString mltService = producer->get("mlt_service"); - if (mltService == "xml" || mltService == "consumer") { - // MLT playlist - mlt_profile prof = producer->get_profile(); - filePropertyMap["progressive"] = QString::number(prof->progressive); - filePropertyMap["colorspace"] = QString::number(prof->colorspace); - filePropertyMap["fps"] = QString::number(mlt_profile_fps(prof)); + // MLT playlist, create producer with blank profile to get real profile info + // TODO: is there an easier way to get this info (original source clip profile) from MLT? + Mlt::Profile *original_profile = new Mlt::Profile(); + Mlt::Producer *tmpProd = new Mlt::Producer(*original_profile, path.toUtf8().constData()); + filePropertyMap["progressive"] = QString::number(original_profile->progressive()); + filePropertyMap["colorspace"] = QString::number(original_profile->colorspace()); + filePropertyMap["fps"] = QString::number(original_profile->fps()); + filePropertyMap["aspect_ratio"] = QString::number(original_profile->sar()); + delete tmpProd; + delete original_profile; } else if (mltService == "avformat") { // Get frame rate @@ -916,10 +932,10 @@ void Render::processFileProperties() filePropertyMap["frame_size"] = QString::number(frame->get_int("width")) + 'x' + QString::number(frame->get_int("height")); filePropertyMap["frequency"] = QString::number(frame->get_int("frequency")); filePropertyMap["channels"] = QString::number(frame->get_int("channels")); - filePropertyMap["aspect_ratio"] = frame->get("aspect_ratio"); + if (!filePropertyMap.contains("aspect_ratio")) filePropertyMap["aspect_ratio"] = frame->get("aspect_ratio"); if (frame->get_int("test_image") == 0) { - if (url.path().endsWith(".mlt") || url.path().endsWith(".westley") || url.path().endsWith(".kdenlive")) { + if (mltService == "xml" || mltService == "consumer") { filePropertyMap["type"] = "playlist"; metadataPropertyMap["comment"] = QString::fromUtf8(producer->get("title")); } else if (frame->get_int("test_audio") == 0) @@ -1079,8 +1095,10 @@ int Render::setProducer(Mlt::Producer *producer, int position) if (producer) delete producer; return -1; } + bool monitorIsActive = false; m_mltConsumer->set("refresh", 0); if (!m_mltConsumer->is_stopped()) { + monitorIsActive = true; m_mltConsumer->stop(); } m_mltConsumer->purge(); @@ -1091,6 +1109,7 @@ int Render::setProducer(Mlt::Producer *producer, int position) if (!producer || !producer->is_valid()) { if (producer) delete producer; producer = m_blackClip->cut(0, 1); + producer->set("id", "black"); } if (!producer || !producer->is_valid()) { @@ -1101,9 +1120,38 @@ int Render::setProducer(Mlt::Producer *producer, int position) emit stopped(); if (position == -1 && producer->get("id") == currentId) position = consumerPosition; if (position != -1) producer->seek(position); + m_fps = producer->get_fps(); int volume = KdenliveSettings::volume(); + if (producer->get_int("_audioclip") == 1) { + // This is an audio only clip, create fake multitrack to apply audiowave filter + Mlt::Tractor *tractor = new Mlt::Tractor(); + Mlt::Producer *color= new Mlt::Producer(*m_mltProfile, "color:red"); + color->set_in_and_out(0, producer->get_out()); + tractor->set_track(*producer, 0); + tractor->set_track(*color, 1); + + Mlt::Consumer xmlConsumer(*m_mltProfile, "xml:audio_hack"); + if (!xmlConsumer.is_valid()) return -1; + xmlConsumer.set("terminate_on_pause", 1); + xmlConsumer.connect(tractor->parent()); + xmlConsumer.run(); + delete tractor; + delete color; + delete producer; + QString playlist = QString::fromUtf8(xmlConsumer.get("audio_hack")); + + Mlt::Producer *result = new Mlt::Producer(*m_mltProfile, "xml-string", playlist.toUtf8().constData()); + Mlt::Filter *filter = new Mlt::Filter(*m_mltProfile, "audiowave"); + result->attach(*filter); + tractor = new Mlt::Tractor(); + tractor->set_track(*result, 0); + delete result; + delete filter; + producer = &(tractor->parent()); + m_mltConsumer->connect(*producer); + } + producer->set("meta.volume", (double)volume / 100); - m_fps = producer->get_fps(); blockSignals(false); m_mltConsumer->connect(*producer); @@ -1114,8 +1162,15 @@ int Render::setProducer(Mlt::Producer *producer, int position) } m_mltProducer = producer; m_mltProducer->set_speed(0); + if (monitorIsActive) startConsumer(); emit durationChanged(m_mltProducer->get_playtime()); - if (m_mltConsumer->start() == -1) { + position = m_mltProducer->position(); + emit rendererPosition(position); + return 0; +} + +void Render::startConsumer() { + if (m_mltConsumer->is_stopped() && m_mltConsumer->start() == -1) { // ARGH CONSUMER BROKEN!!!! KMessageBox::error(qApp->activeWindow(), i18n("Could not create the video preview window.\nThere is something wrong with your Kdenlive install or your driver settings, please fix it.")); if (m_showFrameEvent) delete m_showFrameEvent; @@ -1124,18 +1179,14 @@ int Render::setProducer(Mlt::Producer *producer, int position) m_pauseEvent = NULL; delete m_mltConsumer; m_mltConsumer = NULL; - return -1; + return; } - - position = m_mltProducer->position(); m_mltConsumer->set("refresh", 1); // Make sure the first frame is displayed, otherwise if we change producer too fast // We can crash the avformat producer Mlt::Event *ev = m_mltConsumer->setup_wait_for("consumer-frame-show"); m_mltConsumer->wait_for(ev); delete ev; - emit rendererPosition(position); - return 0; } int Render::setSceneList(QDomDocument list, int position) @@ -1218,7 +1269,6 @@ int Render::setSceneList(QString playlist, int position) blockSignals(true); m_locale = QLocale(); - m_mltProducer = new Mlt::Producer(*m_mltProfile, "xml-string", playlist.toUtf8().constData()); if (!m_mltProducer || !m_mltProducer->is_valid()) { kDebug() << " WARNING - - - - -INVALID PLAYLIST: " << playlist.toUtf8().constData(); @@ -1527,15 +1577,15 @@ void Render::switchPlay(bool play) void Render::play(double speed) { requestedSeekPosition = SEEK_INACTIVE; - if (!m_mltProducer) - return; + if (!m_mltProducer) return; + double current_speed = m_mltProducer->get_speed(); + if (current_speed == speed) return; // if (speed == 0.0) m_mltProducer->set("out", m_mltProducer->get_length() - 1); m_mltProducer->set_speed(speed); - /*if (speed == 0.0) { - m_mltProducer->seek((int) m_framePosition + 1); - m_mltConsumer->purge(); - }*/ - refresh(); + if (m_mltConsumer->is_stopped()) { + m_mltConsumer->start(); + } + if (current_speed == 0 && speed != 0) m_mltConsumer->set("refresh", 1); } void Render::play(const GenTime & startTime) @@ -1596,6 +1646,11 @@ void Render::seekToFrameDiff(int diff) else seek(requestedSeekPosition + diff); } +void Render::refreshIfActive() +{ + if (!m_mltConsumer->is_stopped() && m_mltProducer && m_mltProducer->get_speed() == 0) m_refreshTimer.start(); +} + void Render::doRefresh() { if (m_mltProducer && m_mltProducer->get_speed() == 0) m_refreshTimer.start(); @@ -1608,7 +1663,7 @@ void Render::refresh() return; if (m_mltConsumer) { if (m_mltConsumer->is_stopped()) m_mltConsumer->start(); - m_mltConsumer->purge(); + //m_mltConsumer->purge(); m_mltConsumer->set("refresh", 1); } } @@ -1660,13 +1715,6 @@ void Render::emitFrameUpdated(Mlt::Frame& frame) const uchar* image = frame.get_image(format, width, height); QImage qimage(width, height, QImage::Format_ARGB32_Premultiplied); memcpy(qimage.scanLine(0), image, width * height * 4); - - /*mlt_image_format format = mlt_image_rgb24; - int width = 0; - int height = 0; - const uchar* image = frame.get_image(format, width, height); - QImage qimage(width, height, QImage::Format_RGB888); - memcpy(qimage.bits(), image, width * height * 3);*/ emit frameUpdated(qimage.rgbSwapped()); } @@ -1677,10 +1725,10 @@ void Render::emitFrameNumber() emit rendererPosition(currentPos); if (requestedSeekPosition != SEEK_INACTIVE) { m_mltProducer->seek(requestedSeekPosition); - requestedSeekPosition = SEEK_INACTIVE; if (m_mltProducer->get_speed() == 0) { refresh(); } + requestedSeekPosition = SEEK_INACTIVE; } } @@ -1729,17 +1777,32 @@ void Render::exportCurrentFrame(KUrl url, bool /*notify*/) void Render::showFrame(Mlt::Frame& frame) { - emit rendererPosition((int) m_mltConsumer->position()); - mlt_image_format format = mlt_image_rgb24a; - int width = 0; - int height = 0; - const uchar* image = frame.get_image(format, width, height); - QImage qimage(width, height, QImage::Format_ARGB32_Premultiplied); - memcpy(qimage.scanLine(0), image, width * height * 4); - emit showImageSignal(qimage); - if (analyseAudio) showAudio(frame); - if (sendFrameForAnalysis && frame.get_frame()->convert_image) { - emit frameUpdated(qimage.rgbSwapped()); + int currentPos = m_mltConsumer->position(); + if (currentPos == requestedSeekPosition) requestedSeekPosition = SEEK_INACTIVE; + emit rendererPosition(currentPos); + if (frame.is_valid()) { + mlt_image_format format = mlt_image_rgb24a; + int width = 0; + int height = 0; + const uchar* image = frame.get_image(format, width, height); + QImage qimage(width, height, QImage::Format_ARGB32_Premultiplied); + memcpy(qimage.scanLine(0), image, width * height * 4); + emit showImageSignal(qimage); + if (analyseAudio) showAudio(frame); + if (sendFrameForAnalysis && frame.get_frame()->convert_image) { + emit frameUpdated(qimage.rgbSwapped()); + } + } + emit checkSeeking(); +} + +void Render::slotCheckSeeking() +{ + if (requestedSeekPosition != SEEK_INACTIVE) { + m_mltProducer->seek(requestedSeekPosition); + if (m_mltProducer->get_speed() == 0) { + refresh(); + } } } @@ -1863,6 +1926,7 @@ Mlt::Producer *Render::checkSlowMotionProducer(Mlt::Producer *prod, QDomElement int Render::mltInsertClip(ItemInfo info, QDomElement element, Mlt::Producer *prod, bool overwrite, bool push) { + m_refreshTimer.stop(); if (m_mltProducer == NULL) { kDebug() << "PLAYLIST NOT INITIALISED //////"; return -1; @@ -2012,15 +2076,13 @@ bool Render::mltCutClip(int track, GenTime position) Mlt::Tractor *Render::lockService() { // we are going to replace some clips, purge consumer - QMutexLocker locker(&m_mutex); if (!m_mltProducer) return NULL; + QMutexLocker locker(&m_mutex); if (m_mltConsumer) { - if (!m_mltConsumer->is_stopped()) m_mltConsumer->stop(); m_mltConsumer->purge(); } Mlt::Service service(m_mltProducer->parent().get_service()); if (service.type() != tractor_type) { - kWarning() << "// TRACTOR PROBLEM"; return NULL; } service.lock(); @@ -2094,6 +2156,7 @@ bool Render::mltUpdateClip(Mlt::Tractor *tractor, ItemInfo info, QDomElement ele bool Render::mltRemoveClip(int track, GenTime position) { + m_refreshTimer.stop(); Mlt::Service service(m_mltProducer->parent().get_service()); if (service.type() != tractor_type) { kWarning() << "// TRACTOR PROBLEM"; @@ -4103,6 +4166,7 @@ const QList Render::producersList() Mlt::Producer trackProducer(tt); delete tt; Mlt::Playlist trackPlaylist((mlt_playlist) trackProducer.get_service()); + if (!trackPlaylist.is_valid()) continue; int clipNb = trackPlaylist.count(); for (int i = 0; i < clipNb; i++) { Mlt::Producer *c = trackPlaylist.get_clip(i); @@ -4135,6 +4199,7 @@ void Render::fillSlowMotionProducers() Mlt::Producer trackProducer(tt); delete tt; Mlt::Playlist trackPlaylist((mlt_playlist) trackProducer.get_service()); + if (!trackPlaylist.is_valid()) continue; int clipNb = trackPlaylist.count(); for (int i = 0; i < clipNb; i++) { Mlt::Producer *c = trackPlaylist.get_clip(i); @@ -4505,7 +4570,22 @@ bool Render::getBlackMagicOutputDeviceList(KComboBox *devicelist) } void Render::slotMultiStreamProducerFound(const QString path, QList audio_list, QList video_list, stringMap data) -{ +{ + if (KdenliveSettings::automultistreams()) { + for (int i = 1; i < video_list.count(); i++) { + int vindex = video_list.at(i); + int aindex = 0; + if (i <= audio_list.count() -1) { + aindex = audio_list.at(i); + } + data.insert("video_index", QString::number(vindex)); + data.insert("audio_index", QString::number(aindex)); + data.insert("bypassDuplicate", "1"); + emit addClip(KUrl(path), data); + } + return; + } + int width = 60.0 * m_mltProfile->dar(); int swidth = 60.0 * m_mltProfile->width() / m_mltProfile->height(); if (width % 2 == 1) width++; @@ -4525,13 +4605,12 @@ void Render::slotMultiStreamProducerFound(const QString path, QList audio_l for (int j = 1; j < video_list.count(); j++) { Mlt::Producer multiprod(* m_mltProfile, path.toUtf8().constData()); multiprod.set("video_index", video_list.at(j)); - kDebug()<<"// LOADING: "<setProperty("vindex", video_list.at(j)); groupList << streamFrame; streamFrame->setCheckable(true); - streamFrame->setChecked(false); + streamFrame->setChecked(true); QVBoxLayout *vh = new QVBoxLayout( streamFrame ); QLabel *iconLabel = new QLabel(content); iconLabel->setPixmap(QPixmap::fromImage(thumb)); @@ -4542,7 +4621,7 @@ void Render::slotMultiStreamProducerFound(const QString path, QList audio_l cb->addItem(i18n("Audio stream %1", audio_list.at(k)), audio_list.at(k)); } comboList << cb; - cb->setCurrentIndex(j); + cb->setCurrentIndex(qMin(j, audio_list.count() - 1)); vh->addWidget(cb); } vbox->addWidget(streamFrame);