remove unused docId from result
ClosedPublic

Authored by astippich on Sun, Dec 2, 11:07 AM.

Details

Summary

The doc id in result class is never set and
never read, so remove it. The id can be
retrieved from the document itself.

Test Plan

compiles

Diff Detail

Repository
R293 Baloo
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
astippich created this revision.Sun, Dec 2, 11:07 AM
Restricted Application added projects: Frameworks, Baloo. · View Herald TranscriptSun, Dec 2, 11:07 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
astippich requested review of this revision.Sun, Dec 2, 11:07 AM
bruns added a comment.Sun, Dec 2, 3:12 PM

Addition - the document id is also stored and can be retrieved from the Baloo::Document.

astippich retitled this revision from remove unused doc_id from result to remove unused docId from result.Sun, Dec 2, 5:54 PM
astippich edited the summary of this revision. (Show Details)
ngraham added a subscriber: ngraham.Sun, Dec 2, 6:37 PM

Some of these are public; wouldn't this break ABI compatibility?

bruns added a comment.Sun, Dec 2, 7:28 PM

libKF5BalooEngine is not exported to the outside world.

bruns accepted this revision.Sun, Dec 2, 7:30 PM
This revision is now accepted and ready to land.Sun, Dec 2, 7:30 PM
ngraham accepted this revision.Sun, Dec 2, 7:30 PM

Ah, of course, duh!

This revision was automatically updated to reflect the committed changes.