Update Qt 5 to-do comment in QDom.
The change discussed in the comment doesn't have to be done in a major release if it isn't source-incompatible. Task-number: QTBUG-25103 Change-Id: I50036ab13611871ede01b7b7a17ce4c325476b00 Reviewed-by: Lars Knoll <lars.knoll@nokia.com>bb10
parent
0fc74370b2
commit
7ff5fb610c
|
|
@ -4559,14 +4559,14 @@ void QDomElementPrivate::save(QTextStream& s, int depth, int indent) const
|
|||
QString qName(name);
|
||||
QString nsDecl(QLatin1String(""));
|
||||
if (!namespaceURI.isNull()) {
|
||||
/** ### Qt 5:
|
||||
/** ###
|
||||
*
|
||||
* If we still have QDom, optimize this so that we only declare namespaces that are not
|
||||
* yet declared. We loose default namespace mappings, so maybe we should rather store
|
||||
* the information that we get from startPrefixMapping()/endPrefixMapping() and use them.
|
||||
* Modifications becomes more complex then, however.
|
||||
*
|
||||
* We cannot do this during the Qt 4 series because it would require too invasive changes, and
|
||||
* We cannot do this in a patch release because it would require too invasive changes, and
|
||||
* hence possibly behavioral changes.
|
||||
*/
|
||||
if (prefix.isEmpty()) {
|
||||
|
|
|
|||
Loading…
Reference in New Issue