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

jpd at videolan.org jpd at videolan.org
Fri Jun 12 09:26:55 CEST 2009


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?




More information about the vlc-devel mailing list