我正在关注http://code.google.com/p/dart/wiki/BuildingDartium#Build并尝试以dartium模式构建Release并遇到以下错误:

$ ./dartium_tools/build.py --mode=Release
.
.
.
  CXX(target) out/Release/obj.target/webrtc_video_coding/third_party/webrtc/modules/video_coding/main/source/rtt_filter.o
  CXX(target) out/Release/obj.target/webrtc_video_coding/third_party/webrtc/modules/video_coding/main/source/session_info.o
  CXX(target) out/Release/obj.target/webrtc_video_coding/third_party/webrtc/modules/video_coding/main/source/timestamp_extrapolator.o
third_party/webrtc/modules/video_coding/main/source/session_info.cc: In member function ‘int webrtc::VCMSessionInfo::PrepareForDecode(uint8_t*)’:
third_party/webrtc/modules/video_coding/main/source/session_info.cc:590:8: error: variable ‘previous_lost’ set but not used [-Werror=unused-but-set-variable]
cc1plus: all warnings being treated as errors

make: *** [out/Release/obj.target/webrtc_video_coding/third_party/webrtc/modules/video_coding/main/source/session_info.o] Error 1
make: *** Waiting for unfinished jobs....
Traceback (most recent call last):
  File "./dartium_tools/build.py", line 67, in <module>
    main()
  File "./dartium_tools/build.py", line 64, in main
    [target for (target, _) in targets])
  File "/home/sangeeth/work/g/dartium/src/dartium_tools/utils.py", line 97, in runCommand
    raise Exception('Failed to run command. return code=%s' % p.returncode)
Exception: Failed to run command. return code=2
$

但是,当我查看代码dartium/src/third_party/webrtc/modules/video_coding/main/source/session_info.cc::VCMSessionInfo::PrepareForDecode()时,我看到了以下内容:
int VCMSessionInfo::PrepareForDecode(uint8_t* frame_buffer) {
  int length = SessionLength();
  int real_data_bytes = 0;
  if (length == 0)
      return length;
  bool previous_lost = false;
  PacketIterator it = packets_.begin();
  PacketIterator prev_it = it;
  for (; it != packets_.end(); ++it) {
    bool packet_loss = ((*prev_it).sizeBytes == 0 ||
        !InSequence(it, prev_it));
    if ((*it).bits) {
      if (prev_it != it) {  // Not the first packet.
        uint8_t* ptr_first_byte =
            const_cast<uint8_t*>((*it).dataPtr);

        if (packet_loss) {
          // It is be better to throw away this packet if we are
          // missing the previous packet.
          memset(ptr_first_byte, 0, (*it).sizeBytes);
          previous_lost = true;
          ++packets_not_decodable_;
        } else if ((*it).sizeBytes > 0) {
          // Glue with previous byte.
          // Move everything from [this packet start + 1, end of buffer] one
          // byte to the left.
          uint8_t* ptr_prev_byte =
              const_cast<uint8_t*>((*prev_it).dataPtr) +
              (*prev_it).sizeBytes - 1;
          *ptr_prev_byte = (*ptr_prev_byte) | (*ptr_first_byte);
          memmove(const_cast<uint8_t*>((*it).dataPtr),
                  (*it).dataPtr + 1, (*it).sizeBytes - 1);
          ShiftSubsequentPackets(it, -1);
          (*it).sizeBytes--;
          length--;
          previous_lost = false;
          real_data_bytes += (*it).sizeBytes;
        }
      } else {

        memset(const_cast<uint8_t*>((*it).dataPtr), 0,
               (*it).sizeBytes);
        previous_lost = true;
        ++packets_not_decodable_;
      }
    } else if (packet_loss &&
      (*it).codecSpecificHeader.codec == kRTPVideoH263) {
      // Pad H.263 packet losses with 10 zeros to make it easier
      // for the decoder.
      const int kPaddingLength = 10;
      WebRtc_UWord8 padding_data[kPaddingLength] = {0};
      // Make a copy of the previous packet.
      VCMPacket padding_packet(*it);
      ++padding_packet.seqNum;
      padding_packet.dataPtr = padding_data;
      padding_packet.sizeBytes = kPaddingLength;
      length += InsertPacket(padding_packet, frame_buffer, false, 0);
      previous_lost = true;
    } else {
      real_data_bytes += (*it).sizeBytes;
      previous_lost = false;
    }
    prev_it = it;
  }
  if (real_data_bytes == 0) {
    // Drop the frame since all it contains are zeros.
    for (it = packets_.begin(); it != packets_.end(); ++it)
      (*it).sizeBytes = 0;
    length = 0;
  }
  return length;
}

在许多地方都使用了bool变量previous_lost(设置为falsetrue)。

任何有关如何实现此目标的宝贵意见都将有很大的帮助。

最佳答案

如果不稍后再读取该值,则将bool设置为truefalse是没有意义的-在发布的代码中,确实在多个位置设置了previous_lost的值,但用于任何操作的值都没有设置(例如例如if (previous_lost) ...)。在这种情况下,这就是“使用”的含义:如果读取了变量的值,则使用该变量。

请注意,这(通常)适用于任何类型的变量:如果我将int设置为42,并且以后不使用它,则也不会“使用”它。

在这些情况下,该变量是完全冗余的-由于没有被其他代码使用,因此它没有任何用处。通常,这种警告旨在表示可能会原本打算使用previous_lost的代码被忽略(或被删除)并且需要考虑。另外,变量可能只是多余的,可以完全删除。

关于c++ - Release模式下的Dartium构建错误,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/8813986/

10-09 13:10