On Fri, Mar 31, 2017 at 9:11 PM, Michael Haggerty <mhag...@alum.mit.edu> wrote:
> +void free_ref_cache(struct ref_cache *cache)
> +{
> +       free_ref_entry(cache->root);
> +       free(cache);
> +}

free(NULL) is no-op (and safe). Maybe we should follow the same
pattern for free_ref_cache(). It's really up to you.
-- 
Duy

Reply via email to