Protect EINA_(UN)LIKELY with parenthesis around the expr accepted/tizen/unified/20200707.141004 submit/tizen/20200705.221213
authorWander Lairson Costa <wander.lairson@gmail.com>
Fri, 3 Jul 2020 14:06:03 +0000 (14:06 +0000)
committerJongmin Lee <jm105.lee@samsung.com>
Sun, 5 Jul 2020 21:59:21 +0000 (06:59 +0900)
Without it an expression like !EINA_LIKELY(a && b) expands
!a && b

Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Reviewed-by: Vincent Torri <vincent.torri@gmail.com>
Differential Revision: https://phab.enlightenment.org/D12041

src/lib/eina/eina_types.h

index b93e99f..25ad33a 100644 (file)
 #  define EINA_FORMAT(fmt)
 #  define EINA_CONST
 #  define EINA_NOINSTRUMENT
-#  define EINA_UNLIKELY(exp) exp
-#  define EINA_LIKELY(exp)   exp
+#  define EINA_UNLIKELY(exp) (exp)
+#  define EINA_LIKELY(exp)   (exp)
 #  define EINA_SENTINEL
 #  define EINA_FALLTHROUGH
 #  define EINA_PREFETCH(arg) ((void) (arg))
 # define EINA_FORMAT(fmt)
 # define EINA_CONST
 # define EINA_NOINSTRUMENT
-# define EINA_UNLIKELY(exp) exp
-# define EINA_LIKELY(exp)   exp
+# define EINA_UNLIKELY(exp) (exp)
+# define EINA_LIKELY(exp)   (exp)
 # define EINA_SENTINEL
 # define EINA_FALLTHROUGH
 # define EINA_PREFETCH(arg) ((void) (arg))
 #  define EINA_CONST
 # endif
 # define EINA_NOINSTRUMENT
-# define EINA_UNLIKELY(exp) exp
-# define EINA_LIKELY(exp)   exp
+# define EINA_UNLIKELY(exp) (exp)
+# define EINA_LIKELY(exp)   (exp)
 # define EINA_SENTINEL
 # define EINA_FALLTHROUGH
 # define EINA_PREFETCH(arg) ((void) (arg))
  * processor architectures can then optimize the more likely path.
  * @param[in] exp The expression to be used.
  */
-# define EINA_UNLIKELY(exp) exp
+# define EINA_UNLIKELY(exp) (exp)
 
 /**
  * @def EINA_LIKELY
  * processor architectures can then optimize the more likely path.
  * @param[in] exp The expression to be used.
  */
-# define EINA_LIKELY(exp)   exp
+# define EINA_LIKELY(exp)   (exp)
 
 /**
  * @def EINA_SENTINEL