Comment 7 for bug 540941

Revision history for this message
Jürgen E. Fischer (jef-norbit) wrote :

Right. I overlooked httpresponsecontenttype(0). But removing the offending line is the right thing. The constructor automatically assigns QString::null.

mError( "" ) would assign an empty string, which is just almost the same. There's on check for .isNull() later in the case that would behave different with your proposed change. So either remove the lines, use QString::null instead of 0 or change the .isNull() check.

BTDT: https://trac.osgeo.org/qgis/changeset/13213, https://trac.osgeo.org/qgis/changeset/13215 and https://trac.osgeo.org/qgis/changeset/13299 .