Update jerry-libc unreachable asserts (#1379)

Newer compilers (especially clang) warn (and fail) on
`assert (!"message");` constructs typically used to assert on
unreachable code paths (multiple occurrences in jerry-libc). A more
up-to-date approach is to use `assert (false && "message");`. This
patch applies the pattern to all relevant asserts in jerry-libc.

JerryScript-DCO-1.0-Signed-off-by: Akos Kiss akiss@inf.u-szeged.hu
This commit is contained in:
Akos Kiss
2016-09-29 16:17:46 +02:00
committed by GitHub
parent e2aa714565
commit 509407bfd4
2 changed files with 12 additions and 12 deletions
+10 -10
View File
@@ -253,7 +253,7 @@ libc_printf_write_d_i (FILE *stream, /**< stream pointer */
case LIBC_PRINTF_ARG_LENGTH_TYPE_HIGHL: case LIBC_PRINTF_ARG_LENGTH_TYPE_HIGHL:
{ {
assert (!"unsupported length field L"); assert (false && "unsupported length field L");
} }
} }
@@ -355,13 +355,13 @@ libc_printf_write_u_o_x_X (FILE *stream, /**< stream pointer */
case LIBC_PRINTF_ARG_LENGTH_TYPE_HIGHL: case LIBC_PRINTF_ARG_LENGTH_TYPE_HIGHL:
{ {
assert (!"unsupported length field L"); assert (false && "unsupported length field L");
return; return;
} }
default: default:
{ {
assert (!"unexpected length field"); assert (false && "unexpected length field");
return; return;
} }
} }
@@ -422,7 +422,7 @@ libc_printf_write_u_o_x_X (FILE *stream, /**< stream pointer */
default: default:
{ {
assert (!"unexpected type field"); assert (false && "unexpected type field");
return; return;
} }
} }
@@ -522,7 +522,7 @@ vfprintf (FILE *stream, /**< stream pointer */
if (*format_iter_p == '*') if (*format_iter_p == '*')
{ {
assert (!"unsupported width field *"); assert (false && "unsupported width field *");
} }
// If there is a number, recognize it as field width // If there is a number, recognize it as field width
@@ -535,7 +535,7 @@ vfprintf (FILE *stream, /**< stream pointer */
if (*format_iter_p == '.') if (*format_iter_p == '.')
{ {
assert (!"unsupported precision field"); assert (false && "unsupported precision field");
} }
switch (*format_iter_p) switch (*format_iter_p)
@@ -628,7 +628,7 @@ vfprintf (FILE *stream, /**< stream pointer */
case 'a': case 'a':
case 'A': case 'A':
{ {
assert (!"unsupported double type field"); assert (false && "unsupported double type field");
break; break;
} }
@@ -636,7 +636,7 @@ vfprintf (FILE *stream, /**< stream pointer */
{ {
if (length & LIBC_PRINTF_ARG_LENGTH_TYPE_L) if (length & LIBC_PRINTF_ARG_LENGTH_TYPE_L)
{ {
assert (!"unsupported length field L"); assert (false && "unsupported length field L");
} }
else else
{ {
@@ -659,7 +659,7 @@ vfprintf (FILE *stream, /**< stream pointer */
{ {
if (length & LIBC_PRINTF_ARG_LENGTH_TYPE_L) if (length & LIBC_PRINTF_ARG_LENGTH_TYPE_L)
{ {
assert (!"unsupported length field L"); assert (false && "unsupported length field L");
} }
else else
{ {
@@ -699,7 +699,7 @@ vfprintf (FILE *stream, /**< stream pointer */
case 'n': case 'n':
{ {
assert (!"unsupported type field n"); assert (false && "unsupported type field n");
} }
} }
} }
+2 -2
View File
@@ -134,13 +134,13 @@ fopen (const char *path, /**< file path */
create_if_not_exist = true; create_if_not_exist = true;
if (mode[1] == '+') if (mode[1] == '+')
{ {
assert (!"unsupported mode a+"); assert (false && "unsupported mode a+");
} }
break; break;
} }
default: default:
{ {
assert (!"unsupported mode"); assert (false && "unsupported mode");
} }
} }