<i>static void handle_changed_event(const libvlc_event_t* event, void *param); <br></i>is only one function where <i>event</i> data is readed, and I am not see any problem in it<i>.</i><br><i><br></i><div class="gmail_quote">
2011/11/23 Jean-Paul Saman <span dir="ltr"><<a href="mailto:jpsaman@videolan.org">jpsaman@videolan.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="HOEnZb"><div class="h5">On Tue, Nov 22, 2011 at 1:20 PM, Sergey Radionov <<a href="mailto:rsatom@gmail.com">rsatom@gmail.com</a>> wrote:<br>
> 2011/11/22 Jean-Paul Saman <<a href="mailto:jpsaman@videolan.org">jpsaman@videolan.org</a>><br>
>><br>
>> On Tue, Nov 22, 2011 at 4:31 AM, Sergey Radionov <<a href="mailto:rsatom@gmail.com">rsatom@gmail.com</a>> wrote:<br>
>> > Re-posting. Just to be shure the patch was not forgotten.<br>
>> ><br>
>> > Or it has some problems?<br>
>><br>
>> Patch looks OK for me.<br>
>><br>
>> Could you check the mozilla plugin for the same issue?<br>
><br>
> I've check npapi plugin:<br>
><br>
> static void handle_changed_event(const libvlc_event_t* event, void *param)<br>
> {<br>
>     uint32_t   npcount = 1;<br>
>     NPVariant *npparam = (NPVariant *) NPN_MemAlloc( sizeof(NPVariant) *<br>
> npcount );<br>
><br>
>     VlcPlugin *plugin = (VlcPlugin*)param;<br>
>     switch( event->type )<br>
>     {<br>
> #ifdef LIBVLC120<br>
>         case libvlc_MediaPlayerBuffering:<br>
>             DOUBLE_TO_NPVARIANT(event->u.media_player_buffering.new_cache,<br>
> npparam[0]);<br>
>             break;<br>
> #endif<br>
>         case libvlc_MediaPlayerTimeChanged:<br>
>             DOUBLE_TO_NPVARIANT(<br>
><br>
> static_cast<double>(event->u.media_player_time_changed.new_time),<br>
>                 npparam[0]);<br>
>             break;<br>
>         case libvlc_MediaPlayerPositionChanged:<br>
><br>
> DOUBLE_TO_NPVARIANT(event->u.media_player_position_changed.new_position,<br>
> npparam[0]);<br>
>             break;<br>
>         case libvlc_MediaPlayerSeekableChanged:<br>
><br>
> BOOLEAN_TO_NPVARIANT(event->u.media_player_seekable_changed.new_seekable,<br>
> npparam[0]);<br>
>             break;<br>
>         case libvlc_MediaPlayerPausableChanged:<br>
><br>
> BOOLEAN_TO_NPVARIANT(event->u.media_player_pausable_changed.new_pausable,<br>
> npparam[0]);<br>
>             break;<br>
>         case libvlc_MediaPlayerTitleChanged:<br>
><br>
> BOOLEAN_TO_NPVARIANT(event->u.media_player_title_changed.new_title,<br>
> npparam[0]);<br>
>             break;<br>
>         case libvlc_MediaPlayerLengthChanged:<br>
>             DOUBLE_TO_NPVARIANT(<br>
><br>
> static_cast<double>(event->u.media_player_length_changed.new_length),<br>
>                 npparam[0]);<br>
>             break;<br>
>         default: /* ignore all other libvlc_event_type_t */<br>
>             NPN_MemFree( npparam );<br>
>             return;<br>
>     }<br>
>     plugin->event_callback(event, npparam, npcount, param);<br>
> }<br>
><br>
> so, it have no such functions...<br>
<br>
</div></div>I meant check the npapi plugin for the same issue wrt casting float<br>
and libvlc_time_t correctly, which your ActiveX patch corrected.<br>
<div class="HOEnZb"><div class="h5"><br>
Kind regards,<br>
<br>
Jean-Paul Saman<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br>