c++ - 修复循环变量的 -Wconversion GCC 警告

Question

我想修复现有代码中的编译器警告，并遇到了以下免费函数:

std::uint16_t calculate_crc16(std::vector<unsigned char> const& kData) {
  std::int32_t result{0};

  // This nested loop iterates over each bit of the input data.
  for (unsigned char const& kByte : kData) {
    // TODO(wolters): Refactor to eliminate [-Wconversion] compiler warning.
    for (unsigned char j{1}; j; j <<= 1) {
      if (((result & 1) ? 1 : 0) ^ ((kByte & j) ? 1 : 0)) {
        result >>= 1;
        result ^= 0x9299;
      } else {
        result >>= 1;
      }
    }
  }

  return static_cast<std::uint16_t>(result);
}

TODO 注释下方的行引发以下 GCC v4.7.1 警告:

warning: conversion to 'unsigned char' from 'int' may alter its value [-Wconversion]

如何重构该代码以避免警告？我未能在循环中将 unsigned char 替换为 int，因为这会改变行为。

Answer 1

您可以替换 j <<= 1与

 j = static_cast<unsigned char> (j << 1)

或者甚至

 j = static_cast<unsigned char> ((j << 1)&UCHAR_MAX)

但是，我发现你的代码可读性不太好......也许你可以发表一些评论......