On 16 December 2012 07:49, Brian Paul <span dir="ltr"><<a href="mailto:brian.e.paul@gmail.com" target="_blank">brian.e.paul@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Sat, Dec 15, 2012 at 11:09 PM, Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>> wrote:<br>
> The rather unweildy logic for determining this condition was repeated<br>
> in a large number of places.  This patch consolidates it to a single<br>
> inline function.<br>
><br>
<br>
<br>
</div><div class="im">> @@ -111,4 +113,11 @@ _mesa_PauseTransformFeedback(void);<br>
>  extern void GLAPIENTRY<br>
>  _mesa_ResumeTransformFeedback(void);<br>
><br>
> +static inline bool<br>
> +is_transform_feedback_active_and_unpaused(struct gl_context *ctx)<br>
> +{<br>
> +   return ctx->TransformFeedback.CurrentObject->Active &&<br>
> +      !ctx->TransformFeedback.CurrentObject->Paused;<br>
> +}<br>
<br>
</div>Non-private core Mesa functions should have the _mesa_ prefix.  It's a<br>
long name already.  Maybe use the "xfb" abbreviation?<br>
<br>
The ctx parameter could be const-qualified.<br></blockquote><div><br></div><div><br></div><div>Ok, I'll change this.  Thanks. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
Reviewed-by: Brian Paul <<a href="mailto:brianp@vmware.com">brianp@vmware.com</a>><br>
</blockquote></div><br></div>