www.digitalmars.com         C & C++   DMDScript  

digitalmars.D - #ifdef hell

reply Walter Bright <newshound2 digitalmars.com> writes:
https://www.cqse.eu/en/blog/living-in-the-ifdef-hell/

Another voice on what is wrong with the snarl of #ifdef's.
Oct 29 2015
next sibling parent Walter Bright <newshound2 digitalmars.com> writes:
On 10/29/2015 3:18 PM, Walter Bright wrote:
 https://www.cqse.eu/en/blog/living-in-the-ifdef-hell/

 Another voice on what is wrong with the snarl of #ifdef's.
Reddit: https://www.reddit.com/r/programming/comments/3qo79b/living_in_the_ifdef_hell/
Oct 29 2015
prev sibling parent reply rsw0x <anonymous anonymous.com> writes:
On Thursday, 29 October 2015 at 22:18:11 UTC, Walter Bright wrote:
 https://www.cqse.eu/en/blog/living-in-the-ifdef-hell/

 Another voice on what is wrong with the snarl of #ifdef's.
how does this differ from static if and version(x) hell?
Oct 29 2015
parent reply Walter Bright <newshound2 digitalmars.com> writes:
On 10/29/2015 3:23 PM, rsw0x wrote:
 On Thursday, 29 October 2015 at 22:18:11 UTC, Walter Bright wrote:
 https://www.cqse.eu/en/blog/living-in-the-ifdef-hell/

 Another voice on what is wrong with the snarl of #ifdef's.
how does this differ from static if and version(x) hell?
We advise people not to use static if to do what version(x) eschews, and suggest better alternatives.
Oct 29 2015
next sibling parent reply David Nadlinger <code klickverbot.at> writes:
On Thursday, 29 October 2015 at 23:10:38 UTC, Walter Bright wrote:
 On 10/29/2015 3:23 PM, rsw0x wrote:
 On Thursday, 29 October 2015 at 22:18:11 UTC, Walter Bright 
 wrote:
 https://www.cqse.eu/en/blog/living-in-the-ifdef-hell/

 Another voice on what is wrong with the snarl of #ifdef's.
how does this differ from static if and version(x) hell?
We advise people not to use static if to do what version(x) eschews, and suggest better alternatives.
Ha, I wish! In fact, you are currently arguing for people to write *more* version statements over at the https://github.com/D-Programming-Language/dmd/pull/5231 — David
Oct 29 2015
next sibling parent reply Jonathan M Davis <jmdavisProg gmx.com> writes:
On Thursday, 29 October 2015 at 23:14:26 UTC, David Nadlinger 
wrote:
 On Thursday, 29 October 2015 at 23:10:38 UTC, Walter Bright 
 wrote:
 We advise people not to use static if to do what version(x) 
 eschews, and suggest better alternatives.
Ha, I wish! In fact, you are currently arguing for people to write *more* version statements over at the https://github.com/D-Programming-Language/dmd/pull/5231
The idea is that you get rid of all of the &&ing and ||ing that tends to make #ifdefs really hard to follow and error-prone. And by having each version fully separate, changing one version won't break another. Now, that comes at the cost of forcing you to duplicate a lot of code, which almost everyone other than Walter thinks is worse than the risks associated with merging similar versions together, but as far as I can tell, Walter is completely consistent in his arguments. - Jonathan M Davis
Oct 29 2015
parent Joakim <dlang joakim.fea.st> writes:
On Thursday, 29 October 2015 at 23:55:14 UTC, Jonathan M Davis 
wrote:
 The idea is that you get rid of all of the &&ing and ||ing that 
 tends to make #ifdefs really hard to follow and error-prone. 
 And by having each version fully separate, changing one version 
 won't break another.

 Now, that comes at the cost of forcing you to duplicate a lot 
 of code, which almost everyone other than Walter thinks is 
 worse than the risks associated with merging similar versions 
 together, but as far as I can tell, Walter is completely 
 consistent in his arguments.
I agree that his position is consistent, but I disagree that it implies duplicating all your code. He suggests one possibility in that thread where you can keep code common to OS X and iOS in a single version(Darwin) block, but explicitly stick a "version(OSX) version = Darwin; version(iOS) version = Darwin;" at the top of each druntime file that does so. While that may look like a pimple at the top of each file, it has the virtue that it explicitly, locally documents exactly where this new version is coming from. Note that that's only for druntime and phobos: users are always free to define Darwin for themselves in their build scripts if they want. The alternative in current compilers like gcc is this mess, as he notes on that PR: #define __DBL_MIN_EXP__ (-1021) #define __pentiumpro__ 1 #define __UINT_LEAST16_MAX__ 0xffff #define __ATOMIC_ACQUIRE 2 #define __FLT_MIN__ 1.17549435082228750797e-38F #define __GCC_IEC_559_COMPLEX 2 #define __UINT_LEAST8_TYPE__ unsigned char #define __SIZEOF_FLOAT80__ 12 #define #define __CHAR_BIT__ 8 #define __UINT8_MAX__ 0xff #define __WINT_MAX__ 0xffffffffU #define __ORDER_LITTLE_ENDIAN__ 1234 #define __SIZE_MAX__ 0xffffffffU #define __WCHAR_MAX__ 0x7fffffffL #define __GCC_HAVE_SYNC_COMPARE_AND_SWAP_1 1 #define __GCC_HAVE_SYNC_COMPARE_AND_SWAP_2 1 #define __GCC_HAVE_SYNC_COMPARE_AND_SWAP_4 1 #define __DBL_DENORM_MIN__ ((double)4.94065645841246544177e-324L) #define __GCC_HAVE_SYNC_COMPARE_AND_SWAP_8 1 #define __GCC_ATOMIC_CHAR_LOCK_FREE 2 #define __GCC_IEC_559 2 #define __FLT_EVAL_METHOD__ 2 #define __unix__ 1 #define __GCC_ATOMIC_CHAR32_T_LOCK_FREE 2 #define __UINT_FAST64_MAX__ 0xffffffffffffffffULL #define __SIG_ATOMIC_TYPE__ int #define __DBL_MIN_10_EXP__ (-307) #define __FINITE_MATH_ONLY__ 0 #define __GNUC_PATCHLEVEL__ 0 #define __UINT_FAST8_MAX__ 0xff #define __has_include(STR) __has_include__(STR) #define __DEC64_MAX_EXP__ 385 #define __INT8_C(c) c #define __UINT_LEAST64_MAX__ 0xffffffffffffffffULL #define __SHRT_MAX__ 0x7fff #define __LDBL_MAX__ 1.18973149535723176502e+4932L #define __UINT_LEAST8_MAX__ 0xff #define __GCC_ATOMIC_BOOL_LOCK_FREE 2 #define __UINTMAX_TYPE__ long long unsigned int #define __linux 1 #define __DEC32_EPSILON__ 1E-6DF #define __unix 1 #define __UINT32_MAX__ 0xffffffffU #define __LDBL_MAX_EXP__ 16384 #define __WINT_MIN__ 0U #define __linux__ 1 #define __SCHAR_MAX__ 0x7f #define __WCHAR_MIN__ (-__WCHAR_MAX__ - 1) #define #define __DBL_DIG__ 15 #define __GCC_ATOMIC_POINTER_LOCK_FREE 2 #define __SIZEOF_INT__ 4 #define __SIZEOF_POINTER__ 4 #define __USER_LABEL_PREFIX__ #define __STDC_HOSTED__ 1 #define __LDBL_HAS_INFINITY__ 1 #define __FLT_EPSILON__ 1.19209289550781250000e-7F #define __LDBL_MIN__ 3.36210314311209350626e-4932L #define __STDC_UTF_16__ 1 #define __DEC32_MAX__ 9.999999E96DF #define __INT32_MAX__ 0x7fffffff #define __SIZEOF_LONG__ 4 #define __STDC_IEC_559__ 1 #define __STDC_ISO_10646__ 201304L #define __UINT16_C(c) c #define __DECIMAL_DIG__ 21 #define __gnu_linux__ 1 #define __has_include_next(STR) __has_include_next__(STR) #define __LDBL_HAS_QUIET_NAN__ 1 #define __GNUC__ 5 #define __FLT_HAS_DENORM__ 1 #define __SIZEOF_LONG_DOUBLE__ 12 #define __BIGGEST_ALIGNMENT__ 16 #define __i686 1 #define __DBL_MAX__ ((double)1.79769313486231570815e+308L) #define __INT_FAST32_MAX__ 0x7fffffff #define __DBL_HAS_INFINITY__ 1 #define __DEC32_MIN_EXP__ (-94) #define __INT_FAST16_TYPE__ int #define __LDBL_HAS_DENORM__ 1 #define __DEC128_MAX__ 9.999999999999999999999999999999999E6144DL #define __INT_LEAST32_MAX__ 0x7fffffff #define __DEC32_MIN__ 1E-95DF #define __DBL_MAX_EXP__ 1024 #define __DEC128_EPSILON__ 1E-33DL #define __ATOMIC_HLE_RELEASE 131072 #define __PTRDIFF_MAX__ 0x7fffffff #define __STDC_NO_THREADS__ 1 #define __ATOMIC_HLE_ACQUIRE 65536 #define __LONG_LONG_MAX__ 0x7fffffffffffffffLL #define __SIZEOF_SIZE_T__ 4 #define __SIZEOF_WINT_T__ 4 #define __GCC_HAVE_DWARF2_CFI_ASM 1 #define __GXX_ABI_VERSION 1009 #define __FLT_MIN_EXP__ (-125) #define __i686__ 1 #define __INT_FAST64_TYPE__ long long int #define __DBL_MIN__ ((double)2.22507385850720138309e-308L) #define __DECIMAL_BID_FORMAT__ 1 #define __DEC128_MIN__ 1E-6143DL #define __REGISTER_PREFIX__ #define __UINT16_MAX__ 0xffff #define __DBL_HAS_DENORM__ 1 #define __UINT8_TYPE__ unsigned char #define __NO_INLINE__ 1 #define __i386 1 #define __FLT_MANT_DIG__ 24 #define __VERSION__ "5.2.0" #define #define _STDC_PREDEF_H 1 #define __GCC_ATOMIC_INT_LOCK_FREE 2 #define __FLOAT_WORD_ORDER__ __ORDER_LITTLE_ENDIAN__ #define __STDC_IEC_559_COMPLEX__ 1 #define __INT32_C(c) c #define __DEC64_EPSILON__ 1E-15DD #define __ORDER_PDP_ENDIAN__ 3412 #define __DEC128_MIN_EXP__ (-6142) #define __code_model_32__ 1 #define __INT_FAST32_TYPE__ int #define __UINT_LEAST16_TYPE__ short unsigned int #define unix 1 #define __INT16_MAX__ 0x7fff #define __i386__ 1 #define __SIZE_TYPE__ unsigned int #define __UINT64_MAX__ 0xffffffffffffffffULL #define __INT8_TYPE__ signed char #define __ELF__ 1 #define __FLT_RADIX__ 2 #define __INT_LEAST16_TYPE__ short int #define __LDBL_EPSILON__ 1.08420217248550443401e-19L #define #define __SIG_ATOMIC_MAX__ 0x7fffffff #define __GCC_ATOMIC_WCHAR_T_LOCK_FREE 2 #define __SIZEOF_PTRDIFF_T__ 4 #define __DEC32_SUBNORMAL_MIN__ 0.000001E-95DF #define __pentiumpro 1 #define __INT_FAST16_MAX__ 0x7fffffff #define __UINT_FAST32_MAX__ 0xffffffffU #define __UINT_LEAST64_TYPE__ long long unsigned int #define __FLT_HAS_QUIET_NAN__ 1 #define __FLT_MAX_10_EXP__ 38 #define __LONG_MAX__ 0x7fffffffL #define __DEC128_SUBNORMAL_MIN__ 0.000000000000000000000000000000001E-6143DL #define __FLT_HAS_INFINITY__ 1 #define __UINT_FAST16_TYPE__ unsigned int #define __DEC64_MAX__ 9.999999999999999E384DD #define __CHAR16_TYPE__ short unsigned int #define __PRAGMA_REDEFINE_EXTNAME 1 #define __INT_LEAST16_MAX__ 0x7fff #define __DEC64_MANT_DIG__ 16 #define __INT64_MAX__ 0x7fffffffffffffffLL #define __UINT_LEAST32_MAX__ 0xffffffffU #define __GCC_ATOMIC_LONG_LOCK_FREE 2 #define __INT_LEAST64_TYPE__ long long int #define __INT16_TYPE__ short int #define __INT_LEAST8_TYPE__ signed char #define __STDC_VERSION__ 201112L #define __DEC32_MAX_EXP__ 97 #define __INT_FAST8_MAX__ 0x7f #define __INTPTR_MAX__ 0x7fffffff #define linux 1 #define __LDBL_MANT_DIG__ 64 #define __DBL_HAS_QUIET_NAN__ 1 #define __SIG_ATOMIC_MIN__ (-__SIG_ATOMIC_MAX__ - 1) #define __INTPTR_TYPE__ int #define __UINT16_TYPE__ short unsigned int #define __WCHAR_TYPE__ long int #define __SIZEOF_FLOAT__ 4 #define __UINTPTR_MAX__ 0xffffffffU #define __DEC64_MIN_EXP__ (-382) #define __INT_FAST64_MAX__ 0x7fffffffffffffffLL #define __GCC_ATOMIC_TEST_AND_SET_TRUEVAL 1 #define __FLT_DIG__ 6 #define __UINT_FAST64_TYPE__ long long unsigned int #define __INT_MAX__ 0x7fffffff #define __INT64_TYPE__ long long int #define __FLT_MAX_EXP__ 128 #define __DBL_MANT_DIG__ 53 #define __SIZEOF_FLOAT128__ 16 #define __INT_LEAST64_MAX__ 0x7fffffffffffffffLL #define __GCC_ATOMIC_CHAR16_T_LOCK_FREE 2 #define __DEC64_MIN__ 1E-383DD #define __WINT_TYPE__ unsigned int #define __UINT_LEAST32_TYPE__ unsigned int #define __SIZEOF_SHORT__ 2 #define __LDBL_MIN_EXP__ (-16381) #define __INT_LEAST8_MAX__ 0x7f #define __LDBL_MAX_10_EXP__ 4932 #define __ATOMIC_RELAXED 0 #define __DBL_EPSILON__ ((double)2.22044604925031308085e-16L) #define __UINT8_C(c) c #define __INT_LEAST32_TYPE__ int #define __SIZEOF_WCHAR_T__ 4 #define __UINT64_TYPE__ long long unsigned int #define __INT_FAST8_TYPE__ signed char #define __GNUC_STDC_INLINE__ 1 #define __DBL_DECIMAL_DIG__ 17 #define __STDC_UTF_32__ 1 #define __DEC_EVAL_METHOD__ 2 #define __ORDER_BIG_ENDIAN__ 4321 #define #define __INTMAX_MAX__ 0x7fffffffffffffffLL #define __BYTE_ORDER__ __ORDER_LITTLE_ENDIAN__ #define __FLT_DENORM_MIN__ 1.40129846432481707092e-45F #define __INT8_MAX__ 0x7f #define __UINT_FAST32_TYPE__ unsigned int #define __CHAR32_TYPE__ unsigned int #define __FLT_MAX__ 3.40282346638528859812e+38F #define __INT32_TYPE__ int #define __SIZEOF_DOUBLE__ 8 #define __FLT_MIN_10_EXP__ (-37) #define __INTMAX_TYPE__ long long int #define i386 1 #define __DEC128_MAX_EXP__ 6145 #define __ATOMIC_CONSUME 1 #define __GNUC_MINOR__ 2 #define __UINTMAX_MAX__ 0xffffffffffffffffULL #define __DEC32_MANT_DIG__ 7 #define __DBL_MAX_10_EXP__ 308 #define __LDBL_DENORM_MIN__ 3.64519953188247460253e-4951L #define __INT16_C(c) c #define __STDC__ 1 #define __PTRDIFF_TYPE__ int #define __ATOMIC_SEQ_CST 5 #define __UINT32_TYPE__ unsigned int #define __UINTPTR_TYPE__ unsigned int #define __DEC64_SUBNORMAL_MIN__ 0.000000000000001E-383DD #define __DEC128_MANT_DIG__ 34 #define __LDBL_MIN_10_EXP__ (-4931) #define __SIZEOF_LONG_LONG__ 8 #define __GCC_ATOMIC_LLONG_LOCK_FREE 2 #define __LDBL_DIG__ 18 #define __FLT_DECIMAL_DIG__ 9 #define __UINT_FAST16_MAX__ 0xffffffffU #define __GCC_ATOMIC_SHORT_LOCK_FREE 2 #define __UINT_FAST8_TYPE__ unsigned char #define __ATOMIC_ACQ_REL 4 #define __ATOMIC_RELEASE 3 Note the __linux__, __linux, linux, and __gnu_linux__. :) Of course, that extreme doesn't imply adding a couple more predefined versions to D would automatically land us there, but I can see why he doesn't want to start going down that slippery slope.
Nov 02 2015
prev sibling parent reply Walter Bright <newshound2 digitalmars.com> writes:
On 10/29/2015 4:14 PM, David Nadlinger wrote:
 In fact, you are currently arguing for people to write *more* version
statements
 over at the https://github.com/D-Programming-Language/dmd/pull/5231
I presented several ways to avoid using complex version statements. I might add that over time, I'd been removing #if's and #ifdef's from the dmd front end source code. The results were very satisfactory - the code was easier to read, understand and maintain. It also made running magicport on the code practical. I've been slowly doing the same to the back end.
Oct 29 2015
parent reply Jacob Carlborg <doob me.com> writes:
On 2015-10-30 03:01, Walter Bright wrote:

 I might add that over time, I'd been removing #if's and #ifdef's from
 the dmd front end source code. The results were very satisfactory - the
 code was easier to read, understand and maintain. It also made running
 magicport on the code practical.
The DMD source code contained #ifdef's inside expressions, which is quite a difference compared to "or" and "and" in "static if". -- /Jacob Carlborg
Oct 30 2015
parent Daniel Murphy <yebbliesnospam gmail.com> writes:
On 31/10/2015 12:01 AM, Jacob Carlborg wrote:
 On 2015-10-30 03:01, Walter Bright wrote:

 I might add that over time, I'd been removing #if's and #ifdef's from
 the dmd front end source code. The results were very satisfactory - the
 code was easier to read, understand and maintain. It also made running
 magicport on the code practical.
The DMD source code contained #ifdef's inside expressions, which is quite a difference compared to "or" and "and" in "static if".
Yes, this was the major problem with converting the #ifdefs, not conditions ||ed together. If D allowed oring versions together then version would have been a lot more useful in DDMD.
Nov 02 2015
prev sibling parent anonymous <anonymous anonymous.com> writes:
On Thursday, 29 October 2015 at 23:10:38 UTC, Walter Bright wrote:
 On 10/29/2015 3:23 PM, rsw0x wrote:
 On Thursday, 29 October 2015 at 22:18:11 UTC, Walter Bright 
 wrote:
 https://www.cqse.eu/en/blog/living-in-the-ifdef-hell/

 Another voice on what is wrong with the snarl of #ifdef's.
how does this differ from static if and version(x) hell?
We advise people not to use static if to do what version(x) eschews, and suggest better alternatives.
The same thing can be said about #ifdef. Conventional wisdom in C++ says there's better ways, but the abuse compiles thus continually abused.
Oct 30 2015