[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] lib/rename.c: Conditionally define the out label
From: |
Pádraig Brady |
Subject: |
Re: [PATCH] lib/rename.c: Conditionally define the out label |
Date: |
Mon, 02 Jun 2014 01:20:14 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
On 06/02/2014 01:14 AM, Paul Eggert wrote:
> Pádraig Brady wrote:
>> Well the ; is needed in C++ but optional in C.
>> I was worried about users leaving out the ; by mistake
>> and not noticing in the normal case of compiling in C.
>
> But then the other case should be "# define _GL_UNUSED_LABEL ;", no?
> Otherwise a semicolon would be appended sometimes but not others, which
> cannot be right.
Yes I should have added ; to the else
> My experience is that macros whose expansion ends in ";" tend to cause
> confusion, so I'd rather omit the ";" in both cases.
Yes I agree, let's leave out the trailing ;
thanks,
Pádraig.
- [PATCH] lib/rename.c: Conditionally define the out label, Ben Walton, 2014/06/01
- Re: [PATCH] lib/rename.c: Conditionally define the out label, Pádraig Brady, 2014/06/01
- Re: [PATCH] lib/rename.c: Conditionally define the out label, Pádraig Brady, 2014/06/01
- Re: [PATCH] lib/rename.c: Conditionally define the out label, Paul Eggert, 2014/06/01
- Re: [PATCH] lib/rename.c: Conditionally define the out label, Paul Eggert, 2014/06/01
- Re: [PATCH] lib/rename.c: Conditionally define the out label, Pádraig Brady, 2014/06/01
- Re: [PATCH] lib/rename.c: Conditionally define the out label, Paul Eggert, 2014/06/01
- Re: [PATCH] lib/rename.c: Conditionally define the out label,
Pádraig Brady <=
- Re: [PATCH] lib/rename.c: Conditionally define the out label, Paul Eggert, 2014/06/01
Re: [PATCH] lib/rename.c: Conditionally define the out label, Ben Walton, 2014/06/01
- Re: [PATCH] lib/rename.c: Conditionally define the out label, Paul Eggert, 2014/06/01
- Re: [PATCH] lib/rename.c: Conditionally define the out label, Ben Walton, 2014/06/02
- Re: [PATCH] lib/rename.c: Conditionally define the out label, Paul Eggert, 2014/06/02
- [PATCH] lib/rename.c: rpl_rename - mark the out label as potentially unused, Ben Walton, 2014/06/02
- Re: [PATCH] lib/rename.c: rpl_rename - mark the out label as potentially unused, Pádraig Brady, 2014/06/02
Re: [PATCH] lib/rename.c: Conditionally define the out label, Ben Walton, 2014/06/02