[vlc-devel] [PATCH 2/2] input: add INPUT_SET_VIEWPOINT

[Jean-Baptiste Kempf]

Hello,

On Thu, 10 Nov 2016, at 19:56, Rémi Denis-Courmont wrote:
> You could use separate controls instead of a boolean. Or even a ditch the 
> control and make a function with a separate boolean parameter.

I won't comment on that part.

> But then again, doing this at input thread level will prevent cross-input 
> persistence, which is likely desirable. So that seems too "low level" to
> me.

So far, from what I've seen from 360 videos (and discussed with 360
creators), and the mess those video are, I'd argue we don't want
cross-input persistence for the viewpoint.
In the future, maybe; but for now, I'd argue we need to go back to the
initial viewpoint values set inside the metadata.

Best,

-- 
Jean-Baptiste Kempf -  President
+33 672 704 734