Elijah Newren [off-list ref] writes:

There are two _different_ things named "struct dir_entry" in the codebase:

In the longer term, we should rename such a local type to avoid name
clashes with the global one.  But I of course am OK to leave it
outside the topic to clean it up.

In any case, it is worth saying in the proposed log message why
name-hash cannot use cache.h if we make it include dir.h; it is easy
to do so (i.e. "it has its own 'dir_entry' that is used for other
purpose").

Thanks.