QXmlInputSource is deprecated in qt5.15
Details
autotest ok
Diff Detail
- Repository
- R237 KConfig
- Branch
- port_QXmlStreamReader (branched from master)
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 21652 Build 21670: arc lint + arc unit
Looks good otherwise, this code is pretty well tested too, so I quite trust we aren't breaking it horribly.
Trying to compile the rest of KDE Software against this patch would also be useful.
src/gui/kconfigloader.cpp | ||
---|---|---|
127 | You can trim before toString, this way we save some string allocations. |
src/gui/kconfigloader.cpp | ||
---|---|---|
77 | No need to construct a QString if it's just to check if it's empty. | |
102 | This comparison doesn't need to be onto a converted QString. You can compare against the QStringRef and use QStringRef::compare to make it case insensitive. | |
136 | Use QStringRef::compare() | |
190 | I'd also use QStringRef::compare here |
src/gui/kconfigloader.cpp | ||
---|---|---|
121 | You can use QStringRef::compare() in this if sequence over here too. |
src/gui/kconfigloader.cpp | ||
---|---|---|
87 | This logic is inverted., no? We should be at token document end when reader.atEnd() turns true. In any case on successful parse this condition is met here and false returned. No caller seems to check the return value of ConfigLoaderHandler::parse(), thus nobody noticed? |