diff --git a/iocore/net/quic/Mock.h b/iocore/net/quic/Mock.h index c0d28a3fa6d..eda260c04e4 100644 --- a/iocore/net/quic/Mock.h +++ b/iocore/net/quic/Mock.h @@ -194,13 +194,6 @@ class MockQUICConnectionInfoProvider : public QUICConnectionInfoProvider return NET_VCONNECTION_OUT; } - int - select_next_protocol(SSL *ssl, const unsigned char **out, unsigned char *outlen, const unsigned char *in, - unsigned inlen) const override - { - return SSL_TLSEXT_ERR_OK; - } - bool is_closed() const override { @@ -512,13 +505,6 @@ class MockQUICConnection : public QUICConnection return negotiated_application_name_sv; } - int - select_next_protocol(SSL *ssl, const unsigned char **out, unsigned char *outlen, const unsigned char *in, - unsigned inlen) const override - { - return SSL_TLSEXT_ERR_OK; - } - int _transmit_count = 0; int _retransmit_count = 0; Ptr _mutex; diff --git a/iocore/net/quic/test/test_QUICPacket.cc b/iocore/net/quic/test/test_QUICPacket.cc index 35ffd5ea43c..b9125968e13 100644 --- a/iocore/net/quic/test/test_QUICPacket.cc +++ b/iocore/net/quic/test/test_QUICPacket.cc @@ -249,7 +249,8 @@ TEST_CASE("Sending Packet", "[quic]") CHECK(packet.is_crypto_packet()); packet.store(buf, &len); - CHECK(len == packet.size() - 16); + // See above packet.size() CHECK: packet.size must be greater than 16. + CHECK(len == static_cast(packet.size() - 16)); CHECK(memcmp(buf, expected, len - 16) == 0); } diff --git a/proxy/http3/test/test_QPACK.cc b/proxy/http3/test/test_QPACK.cc index 88fb509b471..167ab7fea22 100644 --- a/proxy/http3/test/test_QPACK.cc +++ b/proxy/http3/test/test_QPACK.cc @@ -221,17 +221,16 @@ output_decoded_headers(FILE *fd, HTTPHdr **headers, uint64_t n) } fprintf(fd, "# stream %" PRIu64 "\n", i + 1); MIMEFieldIter field_iter; - for (MIMEField *field = header_set->iter_get_first(&field_iter); field != nullptr; - field = header_set->iter_get_next(&field_iter)) { - int name_len; - int value_len; + for (auto const &field : *header_set) { + int name_len = 0; + int value_len = 0; Arena arena; - const char *name = field->name_get(&name_len); + const char *name = field.name_get(&name_len); char *lowered_name = arena.str_store(name, name_len); for (int i = 0; i < name_len; i++) { lowered_name[i] = ParseRules::ink_tolower(lowered_name[i]); } - const char *value = field->value_get(&value_len); + const char *value = field.value_get(&value_len); fprintf(fd, "%.*s\t%.*s\n", name_len, lowered_name, value_len, value); } fprintf(fd, "\n");