[lvm-devel] LVM2/daemons/clvmd clvmd.c
Zdenek Kabelac
zkabelac at redhat.com
Tue Oct 26 11:05:06 UTC 2010
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
More information about the lvm-devel
mailing list