[vlc-devel] [PATCH 1/2] Added vout_DisplayScreenSize method (to be verified)

Denis Charmet typx at dinauz.org
Thu Feb 7 17:16:37 CET 2013


Hi,
Le mercredi 30 janvier 2013 à 11:00:57, Poncee Joven a écrit :
> ---
> +void vout_DisplayScreenSize(vout_thread t *vout, int x, int y)
> +{
> +	char * ss[3] = {x, " x " , y};
> +	vout_control_PushString(&vout->p->control, VOUT_CONTROL_OSD_TITLE, ss);
> +}
> +
I fail to see why you send a char** when vout_control_PushString expects
a char * especially when you put inside that char * array to integers.
I expect crashes each time you call that function.
Maybe you want sprintf.

Regards,

-- 
TypX
Le mauvais esprit est un art de vivre



More information about the vlc-devel mailing list