vabaseenc: Move out encoder validation from assertion. (!3051) · Merge requests · GStreamer / gstreamer · GitLab

Snippets Groups Projects

Due to an influx of spam, we have had to impose restrictions on new accounts. Please see this wiki page for instructions on how to get full permissions. Sorry for the inconvenience.

The migration is almost done, at least the rest should happen in the background. There are still a few technical difference between the old cluster and the new ones, and they are summarized in this issue. Please pay attention to the TL:DR at the end of the comment.

Merged Víctor Manuel Jáquez Leal requested to merge vjaquez/gstreamer:va-encoder-no-assert into main 2 years ago

Assertion can be disabled at compilation time. Still it's important to validate it the encoder object was opened by the subclass. This patch removes the assertion and returns if the encoder is open.

Activity

Víctor Manuel Jáquez Leal added Bad Plugins + 1 deleted label 2 years ago

added Bad Plugins + 1 deleted label
He Junyan approved this merge request 2 years ago

approved this merge request
Víctor Manuel Jáquez Leal assigned to @gstreamer-merge-bot 2 years ago

assigned to @gstreamer-merge-bot
Víctor Manuel Jáquez Leal mentioned in commit vjaquez/gstreamer@f9371ccc 2 years ago

mentioned in commit vjaquez/gstreamer@f9371ccc
GStreamer Marge Bot added 11 commits 2 years ago
added 11 commits

f62116e5...7c266306 - 10 commits from branch gstreamer:main

f9371ccc - vabaseenc: Move out encoder validation from assertion.

Compare with previous version
GStreamer Marge Bot merged 2 years ago

merged
GStreamer Release Monkey changed milestone to %1.21.1 2 years ago

changed milestone to %1.21.1

Please register or sign in to reply