Hi, >> +static inline GLuint >Here and below, why 2 spaces between "inline" and "GLuint"?
I have no clue. I suspect it is a scar from some search/replace fiasco over 3 weeks ago. You are the first person who spotted that nit. >> --- a/src/mesa/main/mtypes.h >> +++ b/src/mesa/main/mtypes.h >> @@ -3187,7 +3187,13 @@ struct gl_framebuffer >> * GL_ARB_framebuffer_no_attachments must check for the flag >> _HasAttachments >> * and if GL_FALSE, must then use the values in DefaultGeometry to >> initialize > > * its viewport, scissor and so on (in particular _Xmin, _Xmax, _Ymin and > >- * _Ymax do NOT take into account _HasAttachments being false) > >+ * _Ymax do NOT take into account _HasAttachments being false). To get > >the > >+ * geometry of the framebuffer, the helper functions > Why 2 spaces between "the" and "helper"? No clue. You are again the first person to spot that space. It would be nice to get a "fix these nits I found and get a reviewed by" badge. -Kevin _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
