From 05b8af7a97bd5255980fec2f0514f5bb027f5bec Mon Sep 17 00:00:00 2001 From: Xin Zhuang Date: Wed, 29 May 2019 12:21:50 -0400 Subject: [PATCH] remove use of std::tuple in gzip_filter_test to avoid compilation error in libstdc+++ Signed-off-by: Xin Zhuang --- source/server/server.cc | 6 ++--- .../filters/http/gzip/gzip_filter_test.cc | 27 +++++++++---------- 2 files changed, 16 insertions(+), 17 deletions(-) diff --git a/source/server/server.cc b/source/server/server.cc index 70a763e558a98..15d41ccc07bfe 100644 --- a/source/server/server.cc +++ b/source/server/server.cc @@ -594,15 +594,15 @@ void InstanceImpl::shutdownAdmin() { ServerLifecycleNotifier::HandlePtr InstanceImpl::registerCallback(Stage stage, StageCallback callback) { auto& callbacks = stage_callbacks_[stage]; - return absl::make_unique>(callbacks, callback); + return std::make_unique>(callbacks, callback); } ServerLifecycleNotifier::HandlePtr InstanceImpl::registerCallback(Stage stage, StageCallbackWithCompletion callback) { ASSERT(stage == Stage::ShutdownExit); auto& callbacks = stage_completable_callbacks_[stage]; - return absl::make_unique>(callbacks, - callback); + return std::make_unique>(callbacks, + callback); } void InstanceImpl::notifyCallbacksForStage(Stage stage, Event::PostCb completion_cb) { diff --git a/test/extensions/filters/http/gzip/gzip_filter_test.cc b/test/extensions/filters/http/gzip/gzip_filter_test.cc index fbe8d8a24c24e..740b548a59800 100644 --- a/test/extensions/filters/http/gzip/gzip_filter_test.cc +++ b/test/extensions/filters/http/gzip/gzip_filter_test.cc @@ -144,13 +144,12 @@ class GzipFilterTest : public testing::Test { } void expectValidCompressionStrategyAndLevel( - std::tuple - compression_settings) { + Compressor::ZlibCompressorImpl::CompressionStrategy strategy, absl::string_view strategy_name, + Compressor::ZlibCompressorImpl::CompressionLevel level, absl::string_view level_name) { setUpFilter(fmt::format(R"EOF({{"compression_strategy": "{}", "compression_level": "{}"}})EOF", - std::get<1>(compression_settings), std::get<3>(compression_settings))); - EXPECT_EQ(std::get<0>(compression_settings), config_->compressionStrategy()); - EXPECT_EQ(std::get<2>(compression_settings), config_->compressionLevel()); + strategy_name, level_name)); + EXPECT_EQ(strategy, config_->compressionStrategy()); + EXPECT_EQ(level, config_->compressionLevel()); EXPECT_EQ(5, config_->memoryLevel()); EXPECT_EQ(30, config_->minimumLength()); EXPECT_EQ(28, config_->windowBits()); @@ -211,17 +210,17 @@ TEST_F(GzipFilterTest, DefaultConfigValues) { TEST_F(GzipFilterTest, AvailableCombinationCompressionStrategyAndLevelConfig) { expectValidCompressionStrategyAndLevel( - {Compressor::ZlibCompressorImpl::CompressionStrategy::Filtered, "FILTERED", - Compressor::ZlibCompressorImpl::CompressionLevel::Best, "BEST"}); + Compressor::ZlibCompressorImpl::CompressionStrategy::Filtered, "FILTERED", + Compressor::ZlibCompressorImpl::CompressionLevel::Best, "BEST"); expectValidCompressionStrategyAndLevel( - {Compressor::ZlibCompressorImpl::CompressionStrategy::Huffman, "HUFFMAN", - Compressor::ZlibCompressorImpl::CompressionLevel::Best, "BEST"}); + Compressor::ZlibCompressorImpl::CompressionStrategy::Huffman, "HUFFMAN", + Compressor::ZlibCompressorImpl::CompressionLevel::Best, "BEST"); expectValidCompressionStrategyAndLevel( - {Compressor::ZlibCompressorImpl::CompressionStrategy::Rle, "RLE", - Compressor::ZlibCompressorImpl::CompressionLevel::Speed, "SPEED"}); + Compressor::ZlibCompressorImpl::CompressionStrategy::Rle, "RLE", + Compressor::ZlibCompressorImpl::CompressionLevel::Speed, "SPEED"); expectValidCompressionStrategyAndLevel( - {Compressor::ZlibCompressorImpl::CompressionStrategy::Standard, "DEFAULT", - Compressor::ZlibCompressorImpl::CompressionLevel::Standard, "DEFAULT"}); + Compressor::ZlibCompressorImpl::CompressionStrategy::Standard, "DEFAULT", + Compressor::ZlibCompressorImpl::CompressionLevel::Standard, "DEFAULT"); } // Acceptance Testing with default configuration.