search for: 39c79cf

...gure.ac | 35 +++++++++++++++++++++++++++++++++++ src/alloc.c | 19 +++++++++++++++++++ src/guestfs-internal.h | 33 +++++++++++++++++++++++++++++++++ src/handle.c | 15 +++++++++++++++ 4 files changed, 102 insertions(+) diff --git a/configure.ac b/configure.ac index 39c79cf..7745ab5 100644 --- a/configure.ac +++ b/configure.ac @@ -191,6 +191,41 @@ AC_SYS_LARGEFILE dnl Check sizeof long. AC_CHECK_SIZEOF([long]) +dnl Check if __attribute__((cleanup(...))) works. +dnl XXX It would be nice to use AC_COMPILE_IFELSE here, but gcc just +dnl emits a warning for attributes...

This patch series changes a small part of the library to use __attribute__((cleanup(...))) to automatically free memory when pointers go out of the current scope. In general terms this seems to be a small win although you do have to use it carefully. For functions where you can completely get rid of the "exit code paths", it can simplify things. For a good example, see the