[vlc-devel] [PATCH 00/16] VR/360 navigation, new set

[Rémi Denis-Courmont]

Le keskiviikkona 9. marraskuuta 2016, 19.02.37 EET Thomas Guillem a écrit :
> On Wed, Nov 9, 2016, at 18:58, Rémi Denis-Courmont wrote:
> > Le keskiviikkona 9. marraskuuta 2016, 18.32.54 EET Thomas Guillem a écrit
> > 
> > > Hello, we propose a new set of patches for VR/360 navigation.
> > > 
> > > The first commit changed quite a lot:
> > >  - vout_SetViewpoint() is renamed to vout_UpdateViewpoint(). This new
> > > 
> > > function allow now to increment a viewpoint to the one that is currently
> > > used by the vout.
> > 
> > That seems like a step backward to me. Functions for incremental changes
> > instead of proper setters - we have been there done that already several
> 
> There is also a setter.

I don´t see the point in having both. In practice, there either only be one 
user so that atomicity and arbitration are useless, or there will be an upper 
layer to mediate (presumably the input manager), or both.

-- 
Rémi Denis-Courmont
http://www.remlab.net/