[vlc] Snapshot question

[Sharl.Jimh.Tsin]

from the comment,
maybe you can change it for your convenient,but do not commit.

just guess :)

Best regards,
Sharl.Jimh.Tsin (From China **Obviously Taiwan INCLUDED**)



2011/1/6 Robert Jędrzejczyk <robert at prog.olsztyn.pl>:
> Hello,
>
> I have question: why VideoLAN has hard coded PNG format in function:
>
> int
> libvlc_video_take_snapshot( libvlc_media_player_t *p_mi, unsigned num,
>                             const char *psz_filepath,
>                             unsigned int i_width, unsigned int i_height )
> {
>     assert( psz_filepath );
>
>     vout_thread_t *p_vout = GetVout (p_mi, num);
>     if (p_vout == NULL)
>         return -1;
>
>     /* FIXME: This is not atomic. Someone else could change the values,
>      * at least in theory. */
>     var_SetInteger( p_vout, "snapshot-width", i_width);
>     var_SetInteger( p_vout, "snapshot-height", i_height );
>     var_SetString( p_vout, "snapshot-path", psz_filepath );
>     var_SetString( p_vout, "snapshot-format", "png" );
>     var_TriggerCallback (p_vout, "video-snapshot" );
>     return 0;
> }
>
>
>
> I need to create snapshot in JPG format.
>
> I need report this as bug?
>
>
>
> Best Reagrds
>
> Robert Jędrzejczyk
>
> ______________________________________________________
> vlc mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc
>
>