[Glamor] [PATCH 15/34] glamor: Rename a variable to be more descriptive.

Alex Deucher alexdeucher at gmail.com
Fri Feb 28 10:02:10 PST 2014


From: Eric Anholt <eric at anholt.net>

The "valid_" prefix was used above to describe our allocation that
gets reused multiple times, which is totally unrelated.

Ported from Eric's xserver glamor tree.

Signed-off-by: Eric Anholt <eric at anholt.net>
Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
---
 src/glamor_fill.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/glamor_fill.c b/src/glamor_fill.c
index 634dbe1..1c0b393 100644
--- a/src/glamor_fill.c
+++ b/src/glamor_fill.c
@@ -230,16 +230,17 @@ _glamor_solid_boxes(PixmapPtr pixmap, BoxPtr box, int nbox, float *color)
 
 	while(nbox) {
 		int box_cnt, i;
-		float *valid_vertices;
-		valid_vertices = pvertices;
+		float *next_box;
+
+		next_box = pvertices;
 		box_cnt = nbox > valid_nbox ? valid_nbox : nbox;
 		for (i = 0; i < box_cnt; i++) {
 			glamor_set_normalize_vcoords(pixmap_priv, xscale, yscale,
 						     box[i].x1, box[i].y1,
 						     box[i].x2, box[i].y2,
 						     glamor_priv->yInverted,
-						     valid_vertices);
-			valid_vertices += 4*2;
+						     next_box);
+			next_box += 4 * 2;
 		}
 		if (box_cnt == 1)
 			dispatch->glDrawArrays(GL_TRIANGLE_FAN, 0, box_cnt * 4);
-- 
1.8.3.1



More information about the Glamor mailing list