Fix places that trigger no-unused-lambda-capture

webrtc/video/end_to_end_tests.cc

 /*
  *  Copyright (c) 2013 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 #include <algorithm>
@@ skipping 4260 matching lines @@
 
       return SEND_PACKET;
     }
 
     rtc::Optional<uint16_t> last_observed_sequence_number_ GUARDED_BY(crit_);
     rtc::Optional<uint32_t> last_observed_timestamp_ GUARDED_BY(crit_);
     size_t num_flexfec_packets_sent_ GUARDED_BY(crit_);
     rtc::CriticalSection crit_;
   } observer;
 
-  constexpr int kFrameMaxWidth = 320;
-  constexpr int kFrameMaxHeight = 180;
-  constexpr int kFrameRate = 15;
+  // These would have been declared as constexpr, but then some compilers
+  // require them to be captured in the lambda, and other compilers complain
+  // about no-ununused-lambda-capture. Keeping them as normal variables was
+  // the easiest work-around.
+  int kFrameMaxWidth = 320;
+  int kFrameMaxHeight = 180;
+  int kFrameRate = 15;

[kwiberg-webrtc, 2017/08/23 11:35:59]

const?

[eladalon, 2017/08/23 11:42:53]

Similarly.

 
   Call::Config config(event_log_.get());
 
   std::unique_ptr<test::PacketTransport> send_transport;
   std::unique_ptr<test::PacketTransport> receive_transport;
   std::unique_ptr<VideoEncoder> encoder;
 
   task_queue_.SendTask([&]() {
     CreateCalls(config, config);
 
@@ skipping 718 matching lines @@
     std::unique_ptr<VideoEncoder> encoder_;
     std::unique_ptr<VideoDecoder> decoder_;
     rtc::CriticalSection crit_;
     int recorded_frames_ GUARDED_BY(crit_);
   } test(this);
 
   RunBaseTest(&test);
 }
 
 }  // namespace webrtc