diff options
| author | Elijah Newren <newren@gmail.com> | 2018-08-15 19:54:10 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2018-08-15 20:52:09 +0200 |
| commit | 1076f1e454dae338cee8e48f3e69b58181e5e10a (patch) | |
| tree | 0b52cb2a3051019850000e0474055ec2360e1cf0 /packfile.h | |
| parent | compat/precompose_utf8.h: use more common include guard style (diff) | |
| download | git-1076f1e454dae338cee8e48f3e69b58181e5e10a.tar.xz git-1076f1e454dae338cee8e48f3e69b58181e5e10a.zip | |
Remove forward declaration of an enum
According to http://c-faq.com/null/machexamp.html, sizeof(char*) !=
sizeof(int*) on some platforms. Since an enum could be a char or int
(or long or...), knowing the size of the enum thus is important to
knowing the size of a pointer to an enum, so we cannot just forward
declare an enum the way we can a struct. (Also, modern C++ compilers
apparently define forward declarations of an enum to either be useless
because the enum was defined, or require an explicit size specifier, or
be a compilation error.)
Helped-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to '')
| -rw-r--r-- | packfile.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/packfile.h b/packfile.h index cc7eaffe1b..fa36c473ad 100644 --- a/packfile.h +++ b/packfile.h @@ -1,12 +1,12 @@ #ifndef PACKFILE_H #define PACKFILE_H +#include "cache.h" #include "oidset.h" /* in object-store.h */ struct packed_git; struct object_info; -enum object_type; /* * Generate the filename to be used for a pack file with checksum "sha1" and |