- use .isEmpty() instead of == QLatin1String( "" ) to check for
empty strings
- use .clear() instead of = QLatin1String( "" ) to empty a string
- remove unnecessary QString initializations
This PR adds (optional) synchronization of the master password
with the OS password manager (AKA wallet/keychain).
A set of new menu items has been added in the options ->
authentication -> utilities to manage the new behavior.
Notifications are handled by the message bar unless the
password r/w operation is triggered from a modal dialog,
in this case the notifications will be routed through
the recently exposed QgisApp::showSystemNotification
that uses the OS tray notifications.
This new feature requires libqt5keychain, and was tested
with v. 0.5+
* Remove deprecated Qgis::WKBType and API cleanup
Renames QgsWKBTypes to QgsWkbTypes
Replaces usage of the enums:
* Qgis::WKBType with QgsWkbTypes::Type
* Qgis::GeometryType with QgsWkbTypes::GeometryType
Their values should be forward compatible (a fact that was already
explited up to now by casting between the types)
Renames some SSLxxx to SslXxx and URIxxx to UriXxx
* Fix build warnings and simplify type handling
* Add a fixer to rewrite imports
* The forgotten rebase conflictThe forgotten rebase conflicts
* QgsDataSourcURI > QgsDataSourceUri
* QgsWKBTypes > QgsWkbTypes
* Qgis.WKBGeom > QgsWkbTypes.Geom
* Further python fixes
* Guess what... Qgis::wkbDimensions != QgsWkbTypes::wkbDimensions
* Fix tests
* Python 3 updates
* [travis] pull request caching cannot be disabled
so at least use it in r/w mode
* Fix python3 print in plugins
QString methods
Using single quotes is a significant performance boost. Rough
benchmarks indicate the QString single quote methods take
about 15% of the time the double quote variants take.
qDeleteAll( container.values() ). The values() call isn't
needed and requires building a QList of values first.
Thanks clazy - keep the good optimisations coming!
Rationale:
- there was a lot of large objects passed by value, so potentially
there's a speed bump from this
- even for implicitly shared classes like QString/QList there's still
a (small) cost for copying the objects when there's no reason to
- it's the right thing to do!
> QSslCertificate::subjectInfo() and QSslCertificate::issuerInfo() return a
> QStringList instead of a QString in Qt5. It's pretty common for a certificate
> to contain more than entry of a specific type, but in Qt 4 the API only let
> you access the first one.
[Source](https://blogs.kde.org/2012/04/14/whats-new-qt-5-ssl)
This fix adds macros to apply the Qt4 behavior on Qt5 code. Obviously it would
be better the other way round.
