[systemd-devel] [PATCH] use static_assert instead of assert_cc

Zbigniew Jędrzejewski-Szmek zbyszek at in.waw.pl
Sat Dec 29 08:04:23 PST 2012


On Sat, Dec 29, 2012 at 02:10:27AM +0100, Thomas H.P. Andersen wrote:
> Another item from the todo
Hi,

the patch looks great, but it raises compatibility problems. AFAICT,
required gcc version to raised to 4.6. Debian/squeeze has 4.4. Also,
clang has _Static_assert, not static_assert. I think that a compile
time check is in order. Support for old gcc versions is less
important, so it might be enough to document the requirement in
README, and then maybe provide a fallback if people complain. OTOH
clang compilation support is useful (and used), so it should be kept.

Zbyszek


More information about the systemd-devel mailing list