Merge lp://qastaging/~mhr3/libzeitgeist/search-with-relevancies into lp://qastaging/libzeitgeist

Proposed by Michal Hruby
Status: Merged
Merged at revision: 223
Proposed branch: lp://qastaging/~mhr3/libzeitgeist/search-with-relevancies
Merge into: lp://qastaging/libzeitgeist
Diff against target: 577 lines (+300/-19)
5 files modified
bindings/zeitgeist-1.0.gi (+66/-0)
bindings/zeitgeist-1.0.metadata (+4/-0)
bindings/zeitgeist-1.0.vapi (+37/-12)
src/zeitgeist-index.c (+170/-5)
src/zeitgeist-index.h (+23/-2)
To merge this branch: bzr merge lp://qastaging/~mhr3/libzeitgeist/search-with-relevancies
Reviewer Review Type Date Requested Status
Siegfried Gevatter Approve
Review via email: mp+92553@code.qastaging.launchpad.net

Description of the change

Adds the new search_with_relevancies method. Regenerated vapi.

To post a comment you must log in.
Revision history for this message
Siegfried Gevatter (rainct) wrote :

No tests? :(

> The #ZeitgeistResultType determining the sort order.

two spaces in there

> The #GAsyncResult you received in the #GAsyncReadyCallback you passed
> to zeitgeist_index_search()

_with_relevancies

> The total hit count of the query will be avilable via the returned

avilable

> often be bigger than actual number of events held in the result set, which

s/than/than the/

> is limited by the @num_events paramter passed to

parameter

review: Needs Fixing
223. By Michal Hruby

Fix typos

Revision history for this message
Siegfried Gevatter (rainct) wrote :

Nice.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches