1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Simplify code in QSpdyProtocolHandler::parseHttpHeaders()

Browse Source 
Instead of checking for the presence of a NUL,
then splitting the QByteArray at NUL separators
just to join the parts with different binders,
simply use replace('\0', ...).

Change-Id: I0e0453b80f63cffce4a0ff152120ece754211166
Reviewed-by: Lars Knoll <lars.knoll@theqtcompany.com>
This commit is contained in:
Marc Mutz 2016-04-22 10:13:31 +02:00
parent db27f3d5a7
commit 7ba9863e32
1 changed files with 1 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/network/access/qspdyprotocolhandler.cpp
View File

@ -935,19 +935,7 @@ void QSpdyProtocolHandler::parseHttpHeaders(char flags, const QByteArray &frameD
} else if (name == "content-length") {
httpReply->setContentLength(value.toLongLong());
} else {
if (value.contains('\0')) {
QList<QByteArray> values = value.split('\0');
QByteArray binder(", ");
if (name == "set-cookie")
binder = "\n";
value.clear();
Q_FOREACH (const QByteArray& ivalue, values) {
if (value.isEmpty())
value = ivalue;
else
value += binder + ivalue;
}
}
value.replace('\0', name == "set-cookie" ? "\n" : ", ");
httpReply->setHeaderField(name, value);
}
}