<div dir="ltr"><div>Yes, it's for the patch 13/13. I explained in the cover letter (PATCH 00/13) but I should have explained it in this commit message too.</div><div><br></div><div>This solution is the simpler and the more understandable for me. I tried to do it without it but it was really too confusing.</div>
<div><br></div><div>The thing I don't like about that patch is that it adds an other mutex/cond couple on one port. (there are 3 mutex for the output port if you have hwbuf activated)</div><div>The more you have mutex, bigger are the chance of a deadlook, although here it feels safe.</div>
<div><br></div><div>Maybe we should have only one mutex/cond for one port and use pointers of mutex/cond for FIFO and COUNTER.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-06-29 23:00 GMT+02:00 Martin Storsjö <span dir="ltr"><<a href="mailto:martin@martin.st" target="_blank">martin@martin.st</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Thu, 26 Jun 2014, Thomas Guillem wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Add OMX_BUFFER_COUNT_* macro: Increment count when we send a buffer to OMX<br>
(FillThisBuffer/<u></u>EmptyThisBuffer). Decrement count on<br>
FillBufferDone/EmptyBufferDone callback. On FreeBuffers, wait for buffer count<br>
to be zero.<br>
</blockquote>
<br></div>
Why? To me, this just makes the code much more complicated. Is this required for patch 13/13 in some way? Is it impossible to implement patch 13/13 without this?<br>
<br>
A commit message should not only explain _what_ you do, but more importantly, why (in case that is not obvious, which it is not here).<br>
<br>
// Martin<br>
______________________________<u></u>_________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/<u></u>listinfo/vlc-devel</a><br>
</blockquote></div><br></div>

<br>
<p><span style="color:rgb(136,136,136);font-family:arial,sans-serif;line-height:18.1875px;background-color:rgb(255,255,255)">This email and any files transmitted with it are confidential and are intended solely for the use of the individual or entity to which they are addressed. Access to this e-mail by anyone else is unauthorised. If you are not the intended recipient, any disclosure, copying, distribution or any action taken or omitted to be taken in reliance on it, is prohibited. E-mail messages are not necessarily secure. Archos does not accept responsibility for any changes made to this message after it was sent.</span></p>