Re: [Mesa-dev] [PATCH 0/6] radeon: Plug some memory leaks

2013-11-06 Thread Tom Stellard
These look good, but the indentation seems wrong in patches 2 through 6. -Tom On Wed, Nov 06, 2013 at 10:36:49AM -0600, Aaron Watry wrote: I decided to have some fun and hooked valgrind up to my 7850 while running a few OpenCL tests in piglit. This is the first batch of fixes. Aaron Watry

Re: [Mesa-dev] [PATCH 0/6] radeon: Plug some memory leaks

2013-11-06 Thread Aaron Watry
On Wed, Nov 6, 2013 at 12:15 PM, Tom Stellard t...@stellard.net wrote: These look good, but the indentation seems wrong in patches 2 through 6. An artifact of expanding tabs to 4 spaces in the IDE... Although I'd argue that patch 6 is correct given that the portion of the file affected already