[vlc-devel] [PATCH] Enable aspectratio parameter in mozillaplugin

[jpd at videolan.org]

On Fri, Jun 12, 2009 at 09:02:23AM +0200, R??mi Denis-Courmont wrote:
> This is a generic problem with libvlc vout handling (and aout also).
> 
> IMHO, instead of looking for any single vout, it should:
>  - Apply the setting to all existing vouts for the input,
>  - Store the setting in the appropriate input variable for later vouts,
>  - Not return an exception when there are no vouts.
> 
> This would ensure that the API works regardless of the number of
> outputs and make it a less racy and easier to use. I've been meaning
> to do this in 1.1, but I haven't had time this far. Help welcome.

I believe Sigmund was looking for such a thing recently, if for
different settings (screen positioning). I don't know much about vouts,
so some questions:

- Does this need an extra layer, or is there already an appropriate
  place to do this? Setting a configuration variable means it'll persist
  to the next run, too, which may be overkill or even not what is
  desired at all.

- Some people really don't want their settings to persist. How would
  the difference be communicated? Additional flag? Different API?

- It seems to me that the simplest solution to the current setup,
  javascript, is to add a callback triggered for every new item in
  the playlist. That still requires javascript but at least doesn't
  require polling for new items. How much trouble is adding that?