This revision was automatically updated to reflect the committed changes.
Closed by commit R293:4f131981493b: Use propertymap to store properties in
Baloo::Result (authored by bruns).
CHANGED PRIOR TO COMMIT
https://phabricator.kde.org/D19088?vs=51887=67815#toc
REPOSITORY
R293 Baloo
astippich added a comment.
ping @bruns
are you going to land this series?
REPOSITORY
R293 Baloo
BRANCH
propertymap
REVISION DETAIL
https://phabricator.kde.org/D19088
To: bruns, #baloo, #frameworks, ngraham, poboiko, astippich
Cc: kde-frameworks-devel, hurikhan77, lots0logs,
ngraham accepted this revision.
REPOSITORY
R293 Baloo
BRANCH
propertymap
REVISION DETAIL
https://phabricator.kde.org/D19088
To: bruns, #baloo, #frameworks, ngraham, poboiko, astippich
Cc: kde-frameworks-devel, lots0logs, LeGast00n, fbampaloukas, GB_2, domson,
ashaposhnikov, michaelh,
astippich accepted this revision.
astippich added inline comments.
This revision is now accepted and ready to land.
INLINE COMMENTS
> file.cpp:121
> const QJsonDocument jdoc = QJsonDocument::fromJson(arr);
> -const QVariantMap varMap = jdoc.object().toVariantMap();
> -d->propertyMap
bruns created this revision.
bruns added reviewers: Baloo, Frameworks, ngraham, poboiko, astippich.
Herald added projects: Frameworks, Baloo.
Herald added a subscriber: kde-frameworks-devel.
bruns requested review of this revision.
REVISION SUMMARY
Storing properties directly in a PropertyMap