[PATCH 1/4] Define INLINE_VSYSCALL on all ports
Adhemerval Zanella
adhemerval.zanella@linaro.org
Mon Apr 20 12:56:00 GMT 2015
Hi
On 15-04-2015 19:26, Roland McGrath wrote:
>> * sysdeps/unix/sysv/linux/alpha/sysdep.h [INLINE_VSYSCALL]: Define
>> INLINE_SYSCALL.
>
> The [foo] syntax corresponds to an '#if foo' block. The (foo) syntax is
> what you use when foo is what you touched, whether it's a function,
> variable, macro, type, or whatever. A coherent entry for what the change
> actually does would be:
>
> * sysdeps/unix/sysv/linux/alpha/sysdep.h (INLINE_VSYSCALL):
> New macro; just use INLINE_SYSCALL.
Right, I will change it.
>
> But repeating boilerplate like this across a lot of sysdeps files is
> usually a sign that you're doing it wrong. Why not instead have some
> common place that does:
>
> #ifndef INLINE_VSYSCALL
> # define INLINE_VSYSCALL INLINE_SYSCALL
> #endif
>
> ?
>
That is my idea of a subsequent patch, however seems that a more
comprehensible refactor will be better. I am withdrawing this patch
and I will send another one soon.
More information about the Libc-alpha
mailing list