The definitions for lock() already have a semicolon, resulting in a double semicolon when using the definitions like a normal statement. Signed-off-by: Hannes Reinecke <hare@xxxxxxx> --- libmultipath/lock.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libmultipath/lock.h b/libmultipath/lock.h index 04ef78d..97af0f4 100644 --- a/libmultipath/lock.h +++ b/libmultipath/lock.h @@ -21,11 +21,11 @@ struct mutex_lock { a.depth--; pthread_mutex_unlock(a.mutex) #define lock_cleanup_pop(a) \ fprintf(stderr, "%s:%s(%i) unlock %p depth: %d (%ld)\n", __FILE__, __FUNCTION__, __LINE__, a.mutex, a.depth, pthread_self()); \ - pthread_cleanup_pop(1); + pthread_cleanup_pop(1) #else #define lock(a) a.depth++; pthread_mutex_lock(a.mutex) #define unlock(a) a.depth--; pthread_mutex_unlock(a.mutex) -#define lock_cleanup_pop(a) pthread_cleanup_pop(1); +#define lock_cleanup_pop(a) pthread_cleanup_pop(1) #endif void cleanup_lock (void * data); -- 2.6.6 -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel