Hi,

On Mon, May 27, 2013 at 6:01 PM, Davide Andreoli <d...@gurumeditation.it> wrote:
> 2013/5/27 Davide Andreoli <d...@gurumeditation.it>
>
>> 2013/5/21 Nicolas Aguirre <aguirre.nico...@gmail.com>
>>
>>> Hi there,
>>> please find attached a patch which introduce a new emotion API  :
>>> emotion_module_option_get, which retrieves the parameters set with
>>> emotion_module_option_set. This patch also pass options to the engine.
>>> It's usefull in case of VLC to give specific arguments for example,
>>> the --no-video-title, it's annoying to have the name of the film
>>> displayed at statup.
>>>
>>> I'm not sure this is the best thing we ca do here, but it' doesn't
>>> break the "API" and it does the job, so please review.
>>>
>>
>> I'm not sure either that is the right way to go, but I also really need to
>> remove that annoying
>> title on the movies. Do you have other options to pass or is just the
>> title ?
>>
>> I think that the display of the title should be removed by default, no
>> other engines
>> have that and they should behave all the same way. So it should be easier
>> to just put
>> that option with the other defaults...but:
>>
>> The way we pass options to vlc is wrong. We pass args to libvlc_new(), the
>> docs say:
>>
>> "There is absolutely no warranty or promise of forward, backward and
>> cross-platform compatibility with regards to 
>> libvlc_new()<http://www.videolan.org/developers/vlc/doc/doxygen/html/group__libvlc__core.html#ga915aa5778053d7b52ff9f6ba6e2f7764>arguments.
>>  We recommend that you do not use them, other than when
>> debugging."
>>
>> So I'm not sure that option is supported by other libvlc versions...ideas ?
>>
>
> hmm, I tryed to remove all the args passed to libvlc_new, thus calling
> libvlc_new(0, NULL) ... and everything seems to work fine, someone know
> why we are passing that args?

I think that it originally was needed to make libvlc render in a given
memory buffer, but maybe it's not anymore. It seems that only
libvlc_video_set_callbacks is necessary:

http://www.videolan.org/developers/vlc/doc/doxygen/html/group__libvlc__media__player.html#ga612605f2e5c638d9f4ed59021d714bf0

Anyway, did you set them to NULL on Emotion, or in a test program? I
don't see why not removing them if they are really not needed on
Emotion...

Regards,
--
Rafael Antognolli

------------------------------------------------------------------------------
Try New Relic Now & We'll Send You this Cool Shirt
New Relic is the only SaaS-based application performance monitoring service 
that delivers powerful full stack analytics. Optimize and monitor your
browser, app, & servers with just a few lines of code. Try New Relic
and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_may
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to