http2: inline a couple static variables

It looks cleaner when it's declared where it's defined.

Change-Id: I5006fc086c73e6d4891bc64ff3a8c6b4b17623fe
Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
This commit is contained in:
Mårten Nordheim 2023-07-05 15:12:24 +02:00
parent 8f4e29cc85
commit babdeb7a10
2 changed files with 2 additions and 5 deletions

View File

@ -125,9 +125,6 @@ QUrl urlkey_from_request(const QHttpNetworkRequest &request)
// Since we anyway end up having this in every function definition:
using namespace Http2;
const std::deque<quint32>::size_type QHttp2ProtocolHandler::maxRecycledStreams = 10000;
const quint32 QHttp2ProtocolHandler::maxAcceptableTableSize;
QHttp2ProtocolHandler::QHttp2ProtocolHandler(QHttpNetworkConnectionChannel *channel)
: QAbstractProtocolHandler(channel),
decoder(HPack::FieldLookupTable::DefaultSize),

View File

@ -120,7 +120,7 @@ private:
// the client's preface 24-byte message.
bool waitingForSettingsACK = false;
static const quint32 maxAcceptableTableSize = 16 * HPack::FieldLookupTable::DefaultSize;
inline static const quint32 maxAcceptableTableSize = 16 * HPack::FieldLookupTable::DefaultSize;
// HTTP/2 4.3: Header compression is stateful. One compression context and
// one decompression context are used for the entire connection.
HPack::Decoder decoder;
@ -129,7 +129,7 @@ private:
QHash<QObject *, int> streamIDs;
QHash<quint32, Stream> activeStreams;
std::deque<quint32> suspendedStreams[3]; // 3 for priorities: High, Normal, Low.
static const std::deque<quint32>::size_type maxRecycledStreams;
inline static const std::deque<quint32>::size_type maxRecycledStreams = 10000;
std::deque<quint32> recycledStreams;
// Peer's max frame size (this min is the default value