Skip to content

Commit e210b05

Browse files
committed
Fixes biggerint warn
warning: suggest parentheses around ‘-’ inside ‘>>’ [-Wparentheses] eg: ``` vn[i] = (v[i] << s) | (v[i - 1] >> 16 - s); ``` Signed-off-by: Davide Bettio <davide@uninstall.it>
1 parent 0d5b3d0 commit e210b05

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/libAtomVM/intn.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -305,13 +305,13 @@ static int divmnu16(uint16_t q[], uint16_t r[], const uint16_t u[], const uint16
305305
s = nlz(v[n - 1]) - 16; // 0 <= s <= 15.
306306
uint16_t vn[INTN_DIVMNU_MAX_IN_LEN * (sizeof(intn_digit_t) / sizeof(uint16_t))];
307307
for (i = n - 1; i > 0; i--)
308-
vn[i] = (v[i] << s) | (v[i - 1] >> 16 - s);
308+
vn[i] = (v[i] << s) | (v[i - 1] >> (16 - s));
309309
vn[0] = v[0] << s;
310310

311311
uint16_t un[(INTN_DIVMNU_MAX_IN_LEN * (sizeof(intn_digit_t) / sizeof(uint16_t))) + 1];
312312
un[m] = u[m - 1] >> (16 - s);
313313
for (i = m - 1; i > 0; i--)
314-
un[i] = (u[i] << s) | (u[i - 1] >> 16 - s);
314+
un[i] = (u[i] << s) | (u[i - 1] >> (16 - s));
315315
un[0] = u[0] << s;
316316

317317
for (j = m - n; j >= 0; j--) { // Main loop.
@@ -353,7 +353,7 @@ static int divmnu16(uint16_t q[], uint16_t r[], const uint16_t u[], const uint16
353353
// it and pass it back.
354354
if (r != NULL) {
355355
for (i = 0; i < n; i++)
356-
r[i] = (un[i] >> s) | (un[i + 1] << 16 - s);
356+
r[i] = (un[i] >> s) | (un[i + 1] << (16 - s));
357357
}
358358
return 0;
359359
}

0 commit comments

Comments
 (0)