[PATCH weston] simple-egl: Minor fix ups.

Kristian Høgsberg hoegsberg at gmail.com
Fri Jun 15 07:20:47 PDT 2012


On Wed, Jun 13, 2012 at 05:42:21PM -0600, Scott Moreau wrote:
> 1) Fix typo when initializing variables
> 2) Remove gratuitous glFlush() call
> 3) Localize shader program variable

Thanks, looks good.
Kristian

> ---
>  clients/simple-egl.c |   28 +++++++++++++---------------
>  1 files changed, 13 insertions(+), 15 deletions(-)
> 
> diff --git a/clients/simple-egl.c b/clients/simple-egl.c
> index 4cb5bc7..cceca9e 100644
> --- a/clients/simple-egl.c
> +++ b/clients/simple-egl.c
> @@ -61,7 +61,6 @@ struct window {
>  		GLuint fbo;
>  		GLuint color_rbo;
>  
> -		GLuint program;
>  		GLuint rotation_uniform;
>  
>  		GLuint pos;
> @@ -175,6 +174,7 @@ static void
>  init_gl(struct window *window)
>  {
>  	GLuint frag, vert;
> +	GLuint program;
>  	GLint status;
>  
>  	glViewport(0, 0, window->geometry.width, window->geometry.height);
> @@ -182,31 +182,31 @@ init_gl(struct window *window)
>  	frag = create_shader(window, frag_shader_text, GL_FRAGMENT_SHADER);
>  	vert = create_shader(window, vert_shader_text, GL_VERTEX_SHADER);
>  
> -	window->gl.program = glCreateProgram();
> -	glAttachShader(window->gl.program, frag);
> -	glAttachShader(window->gl.program, vert);
> -	glLinkProgram(window->gl.program);
> +	program = glCreateProgram();
> +	glAttachShader(program, frag);
> +	glAttachShader(program, vert);
> +	glLinkProgram(program);
>  
> -	glGetProgramiv(window->gl.program, GL_LINK_STATUS, &status);
> +	glGetProgramiv(program, GL_LINK_STATUS, &status);
>  	if (!status) {
>  		char log[1000];
>  		GLsizei len;
> -		glGetProgramInfoLog(window->gl.program, 1000, &len, log);
> +		glGetProgramInfoLog(program, 1000, &len, log);
>  		fprintf(stderr, "Error: linking:\n%*s\n", len, log);
>  		exit(1);
>  	}
>  
> -	glUseProgram(window->gl.program);
> +	glUseProgram(program);
>  	
>  	window->gl.pos = 0;
> -	window->gl.pos = 1;
> +	window->gl.col = 1;
>  
> -	glBindAttribLocation(window->gl.program, window->gl.pos, "pos");
> -	glBindAttribLocation(window->gl.program, window->gl.col, "color");
> -	glLinkProgram(window->gl.program);
> +	glBindAttribLocation(program, window->gl.pos, "pos");
> +	glBindAttribLocation(program, window->gl.col, "color");
> +	glLinkProgram(program);
>  
>  	window->gl.rotation_uniform =
> -		glGetUniformLocation(window->gl.program, "rotation");
> +		glGetUniformLocation(program, "rotation");
>  }
>  
>  static void
> @@ -340,8 +340,6 @@ redraw(void *data, struct wl_callback *callback, uint32_t time)
>  	glDisableVertexAttribArray(window->gl.pos);
>  	glDisableVertexAttribArray(window->gl.col);
>  
> -	glFlush();
> -
>  	eglSwapBuffers(window->display->egl.dpy, window->egl_surface);
>  	if (callback)
>  		wl_callback_destroy(callback);
> -- 
> 1.7.7.6
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list