[Mesa-dev] [PATCH] nv50: fix a SIGSEGV with piglit bin/gl-3.1-vao-broken-attrib

Dylan Baker baker.dylan.c at gmail.com
Mon Jul 6 16:55:19 PDT 2015


> >> +      } else
> >> +      if (!vb->buffer) {
> >
> > Should the else and if be on the same line?
> 
> The general style elsewhere is to do it in this weird way. Can't say
> I'm a big fan, but I prefer consistency.
> 
> I'd happily take a change that undid that oddity.
> 

Odd, okay.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150706/27d8c9dd/attachment.sig>


More information about the mesa-dev mailing list