diff options
author | Luke Granger-Brown <git@lukegb.com> | 2020-06-03T01·24+0100 |
---|---|---|
committer | Vincent Ambo <tazjin@google.com> | 2020-06-03T01·24+0100 |
commit | 54b9925b931afbc762e03cced13e23d2b90427c2 (patch) | |
tree | 705dcd6c7b03096ce769d9cdc00eb742ff3b0e8e /tools | |
parent | 76f4e273867b239bed818f569e491470116b6ed6 (diff) |
fix(3p/cgit): Fix compilation under Clang r/878
Clang treats function-like macros "correctly", in that, per the C11 spec: "Each subsequent instance of the function-like macro name followed by a ( [...] is replaced by the replacement list [...]". Additionally, fprintf is also permitted to be defined as a function-like macro rather than as a true function: "Any function declared in a header may be additionally implemented as a function-like macro defined in the header [...]". The specification then suggests surrounding the name of the function in parens to avoid this, which is the technique we use here to avoid the function-like macro being invoked. The other fix here is to use uintptr_t for some arithmetic, since Git is expecting an int as the value here and not a pointer.
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions