mirror of
https://sourceware.org/git/binutils-gdb.git
synced 2024-11-27 03:51:15 +08:00
50500ecfef
This commit fixes a compilation failure in a couple of libctf files due to the use of EOVERFLOW and ENOTSUP, which are not defined when compiling on MinGW. libctf/ChangeLog: PR binutils/25155: * ctf-create.c (EOVERFLOW): If not defined by system header, redirect to ERANGE as a poor man's substitute. * ctf-subr.c (ENOTSUP): If not defined, use ENOSYS instead. This one is how Eli implemented it. I think this implementation has a weakness in the following sense: If other units in libctf start using those constants, we'll get the same error again. Also, I'm wondering whether their use is documented as part of the official libtcf API or not -- users might be writing code that tests for these, and if the system doesn't support them, how would they know what errno code to use in its place. This argues for a having that information in one of libctf's header files. I think it would be nice to have those in ctf-decls.h, but I think we'll need to include <errno.h> in ctf-decls.h if we decide to define those macros there. Rather than second-guess what the CTF developers would prefer, I'm starting by sending Eli's patch, to see what you guys think. Thanks, -- Joel |
||
|---|---|---|
| .. | ||
| aclocal.m4 | ||
| ChangeLog | ||
| config.h.in | ||
| configure | ||
| configure.ac | ||
| ctf-archive.c | ||
| ctf-create.c | ||
| ctf-decl.c | ||
| ctf-decls.h | ||
| ctf-dump.c | ||
| ctf-endian.h | ||
| ctf-error.c | ||
| ctf-hash.c | ||
| ctf-impl.h | ||
| ctf-labels.c | ||
| ctf-link.c | ||
| ctf-lookup.c | ||
| ctf-open-bfd.c | ||
| ctf-open.c | ||
| ctf-qsort_r.c | ||
| ctf-string.c | ||
| ctf-subr.c | ||
| ctf-types.c | ||
| ctf-util.c | ||
| elf.h | ||
| libctf.ver | ||
| Makefile.am | ||
| Makefile.in | ||
| swap.h | ||