Re: [PATCH] test_bitmap_walk: free bitmap with bitmap_free

2015-05-22 Thread Stefan Beller
On Thu, May 21, 2015 at 5:53 PM, Jeff King wrote: > Commit f86a374 (pack-bitmap.c: fix a memleak, 2015-03-30) > noticed that we leak the "result" bitmap. But we should use > "bitmap_free" rather than straight "free", as the former > remembers to free the bitmap array pointed to by the struct. > >

[PATCH] test_bitmap_walk: free bitmap with bitmap_free

2015-05-21 Thread Jeff King
Commit f86a374 (pack-bitmap.c: fix a memleak, 2015-03-30) noticed that we leak the "result" bitmap. But we should use "bitmap_free" rather than straight "free", as the former remembers to free the bitmap array pointed to by the struct. Signed-off-by: Jeff King --- Sorry, I should have noticed thi