# [PATCH] free_ref_entry(): do not trigger reading of loose refs 1 messages from 2012-05-20 to 2012-05-20. Participants: mhagger@alum.mit.edu. Thread: https://gitlist.dev/t/30568 ## mhagger@alum.mit.edu, 2012-05-20 06:49 Subject: [PATCH] free_ref_entry(): do not trigger reading of loose refs Message-ID: <1337496572-11258-1-git-send-email-mhagger@alum.mit.edu> URL: https://gitlist.dev/e/1337496572-11258-1-git-send-email-mhagger%40alum.mit.edu ``` From: Michael Haggerty Do not call get_ref_dir() from within free_ref_entry(), because that triggers the reading of loose refs, only for them to be freed immediately. Signed-off-by: Michael Haggerty --- Fix a silly inefficiency from the lazy-loose changes. refs.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/refs.c b/refs.c index d6bdb47..26c8fcd 100644 --- a/refs.c +++ b/refs.c @@ -259,8 +259,13 @@ static void clear_ref_dir(struct ref_dir *dir); static void free_ref_entry(struct ref_entry *entry) { - if (entry->flag & REF_DIR) - clear_ref_dir(get_ref_dir(entry)); + if (entry->flag & REF_DIR) { + /* + * Do not use get_ref_dir() here, as that might + * trigger the reading of loose refs. + */ + clear_ref_dir(&entry->u.subdir); + } free(entry); } -- 1.7.10 ```