Got it. Will clean this up next week. Thanks for the review.

Sent from my iPhone

On Aug 2, 2013, at 4:14 PM, Alon Zakai <notifications@github.com> wrote:

> In src/library_gl.js:
> 
> > @@ -636,6 +636,13 @@ var LibraryGL = {
> >      for (var i = 0; i < n; i++) {
> >        var id = {{{ makeGetValue('buffers', 'i*4', 'i32') }}};
> >        var buffer = GL.buffers[id];
> > +
> > +      // From spec: "glDeleteBuffers silently ignores 0's and names that do not
> > +      // correspond to existing buffer objects."
> > +      if(buffer === null)
> Shouldn't this be (!buffer)? If it's out of bounds for example it will be undefined, not null
> Per our coding conventions, need space between if and parens
> Open curly brace on same line as iff
> But can put the continue on the same line, so just if (..) continue;
> —
> Reply to this email directly or view it on GitHub.