[vlc-devel] [PATCH 1/3] input core: add status field per elementary stream

Ilkka Ollakka ileoo at videolan.org
Mon Feb 17 07:32:05 CET 2014


Hi,

On Sun, Feb 16, 2014 at 10:51:49AM +0200, Rémi Denis-Courmont wrote:
> Already explained that this makes no sense *yesterday*. Not doing it again.

You had good points for previous patch, and I think in this patch the
thread safety is handled with use of atomic variable.

Otherwise I think this patch has a point that Denis also mentioned, that
currently there isn't other than GUI dialog and error log on failures to
detect issues.

We talked about these patches and it could be that this approach isn't
the most ideal overall, but in that case it's good starting point to
check the issue it focuses on resolving.

-- 
Ilkka Ollakka
No one knows what he can do till he tries.
		-- Publilius Syrus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: Digital signature
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20140217/7b5427a1/attachment.sig>


More information about the vlc-devel mailing list