This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH 1/6] Add INLINE_SYSCALL_RETURN/INLINE_SYSCALL_ERROR_RETURN
- From: Andreas Schwab <schwab at linux-m68k dot org>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: GNU C Library <libc-alpha at sourceware dot org>
- Date: Fri, 14 Aug 2015 14:58:55 +0200
- Subject: Re: [PATCH 1/6] Add INLINE_SYSCALL_RETURN/INLINE_SYSCALL_ERROR_RETURN
- Authentication-results: sourceware.org; auth=none
- References: <20150814120014 dot GA28610 at gmail dot com> <87oaiavy2c dot fsf at igel dot home> <CAMe9rOohDXgjy4Cns1=PfxoyV05jRGJet88darjp43CSErNd+Q at mail dot gmail dot com>
"H.J. Lu" <hjl.tools@gmail.com> writes:
> On Fri, Aug 14, 2015 at 5:44 AM, Andreas Schwab <schwab@linux-m68k.org> wrote:
>> "H.J. Lu" <hjl.tools@gmail.com> writes:
>>
>>> +/* Define a macro which expands inline into the wrapper code for a system
>>> + call and return. */
>>> +#ifndef INLINE_SYSCALL_RETURN
>>> +#define INLINE_SYSCALL_RETURN(name, nr, type, args...) \
>>> + { \
>>> + return (type) INLINE_SYSCALL (name, nr, args); \
>>> + }
>>
>> This should be written as a proper statement that can be followed by a
>> semicolon.
>
> It is done on purpose so that one can write
>
> if ()
> INLINE_SYSCALL_RETURN
> else
> ...
Please reread what I wrote.
Andreas.
--
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."