Path: csiph.com!eternal-september.org!feeder.eternal-september.org!news.eternal-september.org!.POSTED!not-for-mail From: Keith Thompson Newsgroups: comp.lang.c Subject: Re: va_start without the first argument? Date: Tue, 14 Mar 2017 09:18:13 -0700 Organization: None to speak of Lines: 25 Message-ID: References: <7fda2e02-9a6a-411f-9d91-1ad9030ec778@googlegroups.com> <7dc37a16-7135-4e50-accf-591c923908c4@googlegroups.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Injection-Info: mx02.eternal-september.org; posting-host="68145e82dff5cec9d4077a7833a68c0b"; logging-data="25150"; mail-complaints-to="abuse@eternal-september.org"; posting-account="U2FsdGVkX19u8o6optIuOWIf8KnYMK/5" User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) Cancel-Lock: sha1:SSPvTYBRi1Anc+KBeThSqQvVdmY= sha1:wI8aimgfSYQkgXUQ2S8vC51l92k= Xref: csiph.com comp.lang.c:106049 David Brown writes: [...] > #ifdef __GNUC__ > #define IS_COMPILE_TIME_CONSTANT(x) __builtin_constant_p(x) I'd parenthesize this: #define IS_COMPILE_TIME_CONSTANT(x) (__builtin_constant_p(x)) I'm not sure whether it can make a difference, but it's easier to add the parentheses than to prove that they're not needed. > #else > #define IS_COMPILE_TIME_CONSTANT(x) (0) I wouldn't bother with the parentheses here; there are no precedence issues with a single constant. > #endif -- Keith Thompson (The_Other_Keith) kst-u@mib.org Working, but not speaking, for JetHead Development, Inc. "We must do something. This is something. Therefore, we must do this." -- Antony Jay and Jonathan Lynn, "Yes Minister"