Differential D20275

Remove the essentially unused copy of LdapDN
ClosedPublic
Actions

Authored by vkrause on Apr 5 2019, 4:37 PM.

Details

Reviewers
dvratil
Commits
R174:51099265d816: Remove the essentially unused copy of LdapDN
Summary

All this was still used for is basically QByteArray::isEmpty, so use
that directly.

Diff Detail

Repository
R174 KContacts
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
vkrause created this revision.Apr 5 2019, 4:37 PM
Restricted Application added a project: KDE PIM. · View Herald TranscriptApr 5 2019, 4:37 PM
Restricted Application added a subscriber: kde-pim. · View Herald Transcript
vkrause requested review of this revision.Apr 5 2019, 4:37 PM
dvratil accepted this revision.Apr 5 2019, 5:26 PM
dvratil added a subscriber: dvratil.

👍

This revision is now accepted and ready to land.Apr 5 2019, 5:26 PM
Closed by commit R174:51099265d816: Remove the essentially unused copy of LdapDN (authored by vkrause). · Explain WhyApr 6 2019, 3:18 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/CMakeLists.txt (1 line)
DMsrc/ldapdn.cpp (214 lines)
DMsrc/ldapdn_p.h (84 lines)
Msrc/ldif.cpp (10 lines)
Msrc/ldif_p.h (7 lines)

Diff 55571

View Options

src/CMakeLists.txt

Loading...
View Options

src/ldapdn.cpp

Loading...
View Options

src/ldapdn_p.h

Loading...
View Options

src/ldif.cpp

Loading...
View Options

src/ldif_p.h

Loading...