[PATCH weston] screenshooter: silence a warning

Kristian Høgsberg hoegsberg at gmail.com
Wed Jan 8 21:27:30 PST 2014


On Wed, Jan 08, 2014 at 03:39:18PM +0200, ppaalanen at gmail.com wrote:
> From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> 
> screenshooter.c: In function ‘recorder_binding’:
> screenshooter.c:509:5: warning: ‘listener’ may be used uninitialized in
> this function [-Wuninitialized]
> 
> This was not really a problem so far, because the variable was
> uninitialized only in the case where Weston had no outputs.

Patch applied, warning silenced.  Thanks.

Kristian

> Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> ---
>  src/screenshooter.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/screenshooter.c b/src/screenshooter.c
> index 0489f92..26e503c 100644
> --- a/src/screenshooter.c
> +++ b/src/screenshooter.c
> @@ -495,7 +495,7 @@ recorder_binding(struct weston_seat *seat, uint32_t time, uint32_t key, void *da
>  	struct weston_seat *ws = (struct weston_seat *) seat;
>  	struct weston_compositor *ec = ws->compositor;
>  	struct weston_output *output;
> -	struct wl_listener *listener;
> +	struct wl_listener *listener = NULL;
>  	struct weston_recorder *recorder;
>  	static const char filename[] = "capture.wcap";
>  
> -- 
> 1.8.3.2
> 


More information about the wayland-devel mailing list