LVM2/daemons/clvmd clvmd.c
Zdenek Kabelac <[email protected]>
| Newsgroups | dev.linux.lists.lvm-devel |
|---|---|
| Message-ID | <[email protected]> |
Dne 26.10.2010 12:46, Alasdair G Kergon napsal(a): > On Tue, Oct 26, 2010 at 09:57:04AM -0000, zkabelac at sourceware.org wrote: >> +#ifdef __clang__ >> __builtin_unreachable(); >> +#endif > > Please do not scatter #ifdefs like that through the code. > If the standard /* NOTREACHED */ really doesn't work, then hide the > ifdef in a header file and use a #define to make it available precisely > where the compiler supports it, defining it to something empty > elsewhere. Yes - This patch only provides quick compilation hotfix - and I'll prepare better solution later. Zdenek