Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Forward query allocation and aggregate the results #68

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

mmontero
Copy link
Contributor

This will forward the query allocation when received
between the connected listeners and aggregates the
results of each listeners. This helps to indicate
the upstream elements which metas are supported.
For example it avoids v4l2src buffer copy when the
downstream elements doesn't report that video meta
is supported and the pitch is different than width.

*
* Ask @query to the downstream element.
*
* Return: True if the query was successfull, False otherwise.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sucessful

@mmontero mmontero requested review from edisonf and mleivaf October 27, 2020 19:35
This will forward the query allocation when received
between the connected listeners and aggregates the
results of each listeners. This helps to indicate
the upstream elements which metas are supported.
For example it avoids v4l2src buffer copy when the
downstream elements doesn't report that video meta
is supported and the pitch is different than width.
@mmontero mmontero force-pushed the feature/query-allocation branch from 74e4f6a to 1e2c41d Compare October 28, 2020 02:50
@jpeeee
Copy link

jpeeee commented Jan 13, 2023

Why has this pull request not being accepted? Is there a reason why the interpipe should not handle the allocation query in this way? The behaviour is the same as gsttee.c, which makes sense to me, but maybe I am missing something...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants