comments

2 files changed, 7 insertions, 2 deletions

diff --git a/src/mesa/swrast/s_span.c b/src/mesa/swrast/s_span.c
index 5c552e6288..4a562752a8 100644
--- a/src/mesa/swrast/s_span.c
+++ b/src/mesa/swrast/s_span.c
@@ -599,7 +599,6 @@ interpolate_texcoords(GLcontext *ctx, SWspan *span)
                      q += dqdx;
                      w += dwdx;
                   }
-
                }
                else {
                   for (i = 0; i < span->end; i++) {
@@ -817,7 +816,10 @@ interpolate_texcoords(GLcontext *ctx, SWspan *span)
 
 
 /**
- * Fill in the span.varying array from the interpolation values.
+ * Fill in the arrays->attribs[FRAG_ATTRIB_VARx] arrays from the
+ * interpolation values.
+ * XXX since interpolants/arrays are getting uniformed, we might merge
+ * this with interpolate_texcoords(), interpolate_Fog(), etc. someday.
  */
 static INLINE void
 interpolate_varying(GLcontext *ctx, SWspan *span)
diff --git a/src/mesa/swrast/s_span.h b/src/mesa/swrast/s_span.h
index 292679bda1..ff91768001 100644
--- a/src/mesa/swrast/s_span.h
+++ b/src/mesa/swrast/s_span.h
@@ -81,6 +81,9 @@ struct arrays2 {
 typedef struct sw_span_arrays
 {
    /** Per-fragment attributes (indexed by FRAG_ATTRIB_* tokens) */
+   /* XXX someday look at transposing first two indexes for better memory
+    * access pattern.
+    */
    GLfloat attribs[FRAG_ATTRIB_MAX][MAX_WIDTH][4];
 
    /** This mask indicates which fragments are alive or culled */