<div dir="ltr"><div><div>Can you go and manually mark this commit and the "Add entry point for TextureBufferRange" as accepted in Patchwork?  I don't have admin access, and my refactor of the new line caused a rebase.<br><br></div>Thanks.<br><br></div>Laura<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 9, 2015 at 1:13 PM, Laura Ekstrand <span dir="ltr"><<a href="mailto:laura@jlekstrand.net" target="_blank">laura@jlekstrand.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Oh, thanks!  I didn't see the new line there when I read your review.  I will remove it.<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 9, 2015 at 10:45 AM, Anuj Phogat <span dir="ltr"><<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On Mon, Mar 9, 2015 at 9:43 AM, Laura Ekstrand <<a href="mailto:laura@jlekstrand.net" target="_blank">laura@jlekstrand.net</a>> wrote:<br>
> I'm confused which hunk you talking about.  Can you be more specific?<br>
><br>
> On Mon, Mar 9, 2015 at 8:47 AM, Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>> wrote:<br>
>><br>
>> On Wed, Mar 4, 2015 at 4:41 PM, Laura Ekstrand <<a href="mailto:laura@jlekstrand.net" target="_blank">laura@jlekstrand.net</a>><br>
>> wrote:<br>
>> > Adds a useful comment and some whitespace. Fixes an error message.<br>
>> ><br>
>> > v2: Review from Anuj Phogat<br>
>> >    - Split rebase of Tex[ture]Buffer[Range]<br>
>> > ---<br>
>> >  src/mesa/main/teximage.c | 12 ++++++++++--<br>
>> >  1 file changed, 10 insertions(+), 2 deletions(-)<br>
>> ><br>
>> > diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c<br>
>> > index 706c76b..22574bd 100644<br>
>> > --- a/src/mesa/main/teximage.c<br>
>> > +++ b/src/mesa/main/teximage.c<br>
>> > @@ -5354,6 +5354,14 @@ _mesa_TexBufferRange(GLenum target, GLenum<br>
>> > internalFormat, GLuint buffer,<br>
>> >                    buffer);<br>
>> >        return;<br>
>> >     } else {<br>
>> > +<br>
>> > +      /* OpenGL 4.5 core spec (02.02.2015) says in Section 8.9 Buffer<br>
>> > +       * Textures (PDF page 254):<br>
>> > +       *    "If buffer is zero, then any buffer object attached to the<br>
>> > buffer<br>
>> > +       *    texture is detached, the values offset and size are ignored<br>
>> > and<br>
>> > +       *    the state for offset and size for the buffer texture are<br>
>> > reset to<br>
>> > +       *    zero."<br>
>> > +       */<br>
>> >        offset = 0;<br>
>> >        size = 0;<br>
>> >     }<br>
>> > @@ -5382,8 +5390,7 @@ _mesa_TextureBuffer(GLuint texture, GLenum<br>
>> > internalFormat, GLuint buffer)<br>
>> >        bufObj = NULL;<br>
>> ><br>
>> >     /* Get the texture object by Name. */<br>
>> > -   texObj = _mesa_lookup_texture_err(ctx, texture,<br>
>> > -                                     "glTextureBuffer(texture)");<br>
>> > +   texObj = _mesa_lookup_texture_err(ctx, texture, "glTextureBuffer");<br>
>> >     if (!texObj)<br>
>> >        return;<br>
>> ><br>
>> > @@ -5396,6 +5403,7 @@ _mesa_TextureBuffer(GLuint texture, GLenum<br>
>> > internalFormat, GLuint buffer)<br>
>> >                                bufObj, 0, buffer ? -1 : 0,<br>
>> > "glTextureBuffer");<br>
>> >  }<br>
>> ><br>
>> > +<br>
</div></div>I meant this extra new line here. It's a nitpick. Up to you if you<br>
want to keep it.<br>
<div><div><br>
>> >  static GLboolean<br>
>> >  is_renderable_texture_format(struct gl_context *ctx, GLenum<br>
>> > internalformat)<br>
>> >  {<br>
>> This hunk is unnecessary.<br>
>> > --<br>
>> > 2.1.0<br>
>> ><br>
>> > _______________________________________________<br>
>> > mesa-dev mailing list<br>
>> > <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
>> > <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
><br>
><br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>