Fix unwanted type promotion.
authorbatt <batt@38d2e660-2303-0410-9eaa-f027e97ec537>
Tue, 4 Mar 2008 16:21:00 +0000 (16:21 +0000)
committerbatt <batt@38d2e660-2303-0410-9eaa-f027e97ec537>
Tue, 4 Mar 2008 16:21:00 +0000 (16:21 +0000)
git-svn-id: https://src.develer.com/svnoss/bertos/trunk@1161 38d2e660-2303-0410-9eaa-f027e97ec537

cfg/macros.h

index f0520eff74b24d5a145eabe29f994aa2d0496f88..1006fc42e6ac9cfee30ce838aeb04a7953933b4f 100644 (file)
                typeof(a) _a = (a); \
                typeof(b) _b = (b); \
                ASSERT_TYPE_EQUAL(_a, _b); \
-               (_a < _b) ? _a : _b; \
+               /** \
+                * The (typeof(_a)) cast in necessary: \
+                * result type of conditional expressions is \
+                * *NOT* the type of the value returned but \
+                * the type that would be produced if _a and _b \
+                * were mixed in an expression. \
+                * Even in _a and _b are of the same type, \
+                * if mixed in an expression the type will be \
+                * promoted to int! \
+               */ \
+               ((typeof(_a))((_a < _b) ? _a : _b)); \
        })
        #define MAX(a,b) ({ \
                typeof(a) _a = (a); \
                typeof(b) _b = (b); \
                ASSERT_TYPE_EQUAL(_a, _b); \
-               (_a > _b) ? _a : _b; \
+               /** \
+                * The (typeof(_a)) cast in necessary: \
+                * result type of conditional expressions is \
+                * *NOT* the type of the value returned but \
+                * the type that would be produced if _a and _b \
+                * were mixed in an expression. \
+                * Even in _a and _b are of the same type, \
+                * if mixed in an expression the type will be \
+                * promoted to int! \
+               */ \
+               ((typeof(_a))((_a > _b) ? _a : _b)); \
        })
 #else /* !(COMPILER_STATEMENT_EXPRESSIONS && COMPILER_TYPEOF) */
        /* Buggy macros for inferior compilers.  */