[Bug 784911] aggregate: chain_internal function called from two different threads for the same pad

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=784911

Nicolas Dufresne (stormer) <nicolas at ndufresne.ca> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #355559|none                        |reviewed
             status|                            |

--- Comment #10 from Nicolas Dufresne (stormer) <nicolas at ndufresne.ca> ---
Review of attachment 355559:
 --> (https://bugzilla.gnome.org/review?bug=784911&attachment=355559)

Seems fair, though can you update the comment next to the flush lock, so we
remember that it shouldn't be taken from the srcpad stream thread. Also, maybe
a comment somewhere next to chain_internal that it can be called from the
srcpad thread for GAP handling ?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.