On 7 November 2011 17:34, Marek Olšák <mar...@gmail.com> wrote:
> On Tue, Nov 8, 2011 at 1:13 AM, Paul Berry <stereotype...@gmail.com>
> wrote:
> > +/**
> > + * Update gl_transform_feedback_info to reflect this tfeedback_decl.
> > + *
> > + * If an error occurs, the error is reported through linker_error() and
> false
> > + * is returned.
> > + */
> > +bool
> > +tfeedback_decl::store(struct gl_shader_program *prog,
> > + struct gl_transform_feedback_info *info,
> > + unsigned buffer) const
> > +{
> > + if (!this->is_assigned()) {
> > + /* From GL_EXT_transform_feedback:
> > + * A program will fail to link if:
> > + *
> > + * * any variable name specified in the <varyings> array is not
> > + * declared as an output in the geometry shader (if present)
> or
> > + * the vertex shader (if no geometry shader is present);
> > + */
> > + linker_error(prog, "Transform feedback varying %s undeclared.",
> > + this->orig_name);
> > + return false;
> > + }
> > + for (unsigned v = 0; v < this->vector_elements; ++v) {
>
> I think the number of iterations should be this->matrix_columns, not
> this->vector_elements.
>
You're absolutely right. I'll fix this.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
