On 10/23/2014 10:55 AM, Matt Turner wrote:
> On Thu, Oct 23, 2014 at 9:35 AM, Ian Romanick <[email protected]> wrote:
>> This is just a big pile of clean up patches that I accumulated while
>> working on something else.  These have been mostly stable for a few
>> days, and the other work is starting to settle.  I figured I'd send out
>> any easy series that anyone can review before sending out the "real"
>> work.
> 
> The series is
> 
> Reviewed-by: Matt Turner <[email protected]>
> 
> Patch 4 didn't come through, but I found it in your repo. I was
> initially confused by patch 3, so let me explain my understanding and
> you tell me if my review was worth anything :)
> 
> Patch 3 adds a bunch of frees (that will just be calling
> ralloc_free(NULL) initially) for the purpose of moving them into a
> common location in patch 4.

That is correct.  I wonder why patch 4 never showed up...

_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to