diff options
author | Lennart Poettering <lennart@poettering.net> | 2013-10-14 04:59:26 +0200 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2013-10-14 06:11:19 +0200 |
commit | 14bf2c9d375db6a4670bc0ef0e521e35a939a498 (patch) | |
tree | 704fdf5312e54dc1231eecbadc90df3f15b33096 /src/shared/fdset.h | |
parent | 416446221d905b6815175dc4d525d27f8ae43d1b (diff) |
util: allow trailing semicolons on define_trivial_cleanup_func lines
Emacs C indenting really gets confused by these lines if they carry no
trailing semicolon, hence let's make this nicer for good old emacs. The
other macros which define functions already do this too, so let's copy
the scheme here.
Also, let's use an uppercase name for the macro. So far our rough rule
was that macros that are totally not function-like (like this ones,
which define a function) are uppercase. (Well, admittedly it is a rough
rule only, for example function and variable decorators are all
lower-case SINCE THE CONSTANT YELLING IN THE SOURCES WOULD SUCK, and
also they at least got underscore prefixes.) Also, the macros that
define functions that we already have are all uppercase, so let's do the
same here...
Diffstat (limited to 'src/shared/fdset.h')
-rw-r--r-- | src/shared/fdset.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/shared/fdset.h b/src/shared/fdset.h index 6277e464d8..907acd76dd 100644 --- a/src/shared/fdset.h +++ b/src/shared/fdset.h @@ -49,5 +49,5 @@ int fdset_iterate(FDSet *s, Iterator *i); #define FDSET_FOREACH(fd, fds, i) \ for ((i) = ITERATOR_FIRST, (fd) = fdset_iterate((fds), &(i)); (fd) >= 0; (fd) = fdset_iterate((fds), &(i))) -define_trivial_cleanup_func(FDSet*, fdset_free) +DEFINE_TRIVIAL_CLEANUP_FUNC(FDSet*, fdset_free); #define _cleanup_fdset_free_ _cleanup_(fdset_freep) |