https://bugs.kde.org/show_bug.cgi?id=414814

--- Comment #2 from Jonathan Gilbert <c6kargn...@liamekaens.com> ---
I've just poked at the source code to see what's up, and I note the following:

* All of the code involved is found inside src/dialogs/renderwidget.cpp.

* The validation is done by RenderWidget::refreshView, based on ambient
supportedFormats, acodecsList and vcodecsList.

* There is a probable bug in that if acodec fails validation, then it sets the
item background to red, and this line is missing for when f and vcodec fail
validation.

* The ambient list of supported codecs/formats is populated in
RenderWidget::checkCodecs by asking the MLT consumer:
consumer->get_data("vcodec"), consumer->get_data("acodec"),
consumer->get_data("f"). Therefore, if "mkv" isn't in the supported list, it's
possibly an MLT bug, MLT isn't reporting a format it actually supports. I took
a peek at the MLT codebase but I can't see any way to quickly identify how
consumer->get_data("f") gets its values.

* The validation should probably be done within RenderWidget::parseProfiles, as
all of the different methods that involve changes to the profile list funnel
down to this.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to