Skip to content

Commit b930cd8

Browse files
committed
Fix more conversion warnings
1 parent e389dbb commit b930cd8

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

include/boost/decimal/dpd_conversion.hpp

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,7 @@ constexpr auto encode_dpd(std::uint8_t d1, std::uint8_t d2, std::uint8_t d3) ->
227227

228228
for (std::uint16_t i {}; i < 10U; ++i)
229229
{
230-
result |= (result_b[i] << (9 - i));
230+
result |= static_cast<std::uint16_t>(result_b[i] << (9 - i));
231231
}
232232

233233
return result;
@@ -256,49 +256,49 @@ constexpr auto decode_dpd(std::uint32_t dpd_bits, std::uint8_t& d3, std::uint8_t
256256
{
257257
d1 = static_cast<std::uint8_t>((b[0] << 2U) + (b[1] << 1U) + b[2]);
258258
d2 = static_cast<std::uint8_t>((b[3] << 2U) + (b[4] << 1U) + b[5]);
259-
d3 = static_cast<std::uint8_t>(8) + b[9];
259+
d3 = static_cast<std::uint8_t>(8U + b[9]);
260260
}
261261
// 101XX
262262
else if (b[6] == 1U && b[7] == 0U && b[8] == 1U)
263263
{
264264
d1 = static_cast<std::uint8_t>((b[0] << 2U) + (b[1] << 1U) + b[2]);
265-
d2 = static_cast<std::uint8_t>(8) + b[5];
265+
d2 = static_cast<std::uint8_t>(8U + b[5]);
266266
d3 = static_cast<std::uint8_t>((b[3] << 2U) + (b[4] << 1U) + b[9]);
267267
}
268268
// 110XX
269269
else if (b[6] == 1U && b[7] == 1U && b[8] == 0U)
270270
{
271-
d1 = static_cast<std::uint8_t>(8) + b[2];
271+
d1 = static_cast<std::uint8_t>(8U + b[2]);
272272
d2 = static_cast<std::uint8_t>((b[3] << 2U) + (b[4] << 1U) + b[5]);
273273
d3 = static_cast<std::uint8_t>((b[0] << 2U) + (b[1] << 1U) + b[9]);
274274
}
275275
// 11100
276276
else if (b[6] == 1U && b[7] == 1U && b[8] == 1U && b[3] == 0U && b[4] == 0U)
277277
{
278-
d1 = static_cast<std::uint8_t>(8) + b[2];
279-
d2 = static_cast<std::uint8_t>(8) + b[5];
278+
d1 = static_cast<std::uint8_t>(8U + b[2]);
279+
d2 = static_cast<std::uint8_t>(8U + b[5]);
280280
d3 = static_cast<std::uint8_t>((b[0] << 2U) + (b[1] << 1U) + b[9]);
281281
}
282282
// 11101
283283
else if (b[6] == 1U && b[7] == 1U && b[8] == 1U && b[3] == 0U && b[4] == 1U)
284284
{
285-
d1 = static_cast<std::uint8_t>(8) + b[2];
285+
d1 = static_cast<std::uint8_t>(8U + b[2]);
286286
d2 = static_cast<std::uint8_t>((b[0] << 2U) + (b[1] << 1U) + b[5]);
287-
d3 = static_cast<std::uint8_t>(8) + b[9];
287+
d3 = static_cast<std::uint8_t>(8U + b[9]);
288288
}
289289
// 11110
290290
else if (b[6] == 1U && b[7] == 1U && b[8] == 1U && b[3] == 1U && b[4] == 0U)
291291
{
292292
d1 = static_cast<std::uint8_t>((b[0] << 2U) + (b[1] << 1U) + b[2]);
293-
d2 = static_cast<std::uint8_t>(8) + b[5];
294-
d3 = static_cast<std::uint8_t>(8) + b[9];
293+
d2 = static_cast<std::uint8_t>(8U + b[5]);
294+
d3 = static_cast<std::uint8_t>(8U + b[9]);
295295
}
296296
// 11111
297297
else if (b[6] == 1U && b[7] == 1U && b[8] == 1U && b[3] == 1U && b[4] == 1U)
298298
{
299-
d1 = static_cast<std::uint8_t>(8) + b[2];
300-
d2 = static_cast<std::uint8_t>(8) + b[5];
301-
d3 = static_cast<std::uint8_t>(8) + b[9];
299+
d1 = static_cast<std::uint8_t>(8U + b[2]);
300+
d2 = static_cast<std::uint8_t>(8U + b[5]);
301+
d3 = static_cast<std::uint8_t>(8U + b[9]);
302302
}
303303
else
304304
{

0 commit comments

Comments
 (0)