dee

Merge lp://qastaging/~kamstrup/dee/sorted into lp://qastaging/dee

Proposed by Mikkel Kamstrup Erlandsen
Status: Merged
Approved by: Michal Hruby
Approved revision: 344
Merged at revision: 334
Proposed branch: lp://qastaging/~kamstrup/dee/sorted
Merge into: lp://qastaging/dee
Diff against target: 1441 lines (+775/-228)
13 files modified
bindings/python/Dee.py (+6/-0)
src/dee-file-resource-manager.c (+0/-1)
src/dee-filter-model.c (+86/-0)
src/dee-filter.c (+125/-170)
src/dee-filter.h (+5/-0)
src/dee-model.c (+180/-0)
src/dee-model.h (+49/-0)
src/dee-sequence-model.c (+74/-43)
src/dee-serializable-model.c (+70/-0)
tests/test-benchmark.c (+76/-14)
tests/test-model-rows.c (+95/-0)
vapi/Dee-1.0.metadata (+2/-0)
vapi/dee-1.0.vapi (+7/-0)
To merge this branch: bzr merge lp://qastaging/~kamstrup/dee/sorted
Reviewer Review Type Date Requested Status
Michal Hruby (community) Approve
Review via email: mp+87950@code.qastaging.launchpad.net

Description of the change

Adds two new methods to the DeeModel interface: insert_row_sorted() and find_row_sorted(). Adding varargs convenience functions as well. There are linear default impls covering all our classes in DeeSerializableModel. We have optimized tree-based algorithms implemented for DeeSequenceModel and DeeFilterModel.

The Python support is currently broken on some bug in pygobject. I believed that my pygobject patches on https://bugzilla.gnome.org/show_bug.cgi?id=667244 would fix this, but testing unfortunately confirms that it doesn't...

To post a comment you must log in.
lp://qastaging/~kamstrup/dee/sorted updated
342. By Mikkel Kamstrup Erlandsen

Add a benchmark for sorted inserts straight on a DeeSequenceModel

343. By Mikkel Kamstrup Erlandsen

Include dee_model_insert_sorted() varargs variant in .vapi

344. By Mikkel Kamstrup Erlandsen

Add missing gtk-doc and annotations for gir/vapi

Revision history for this message
Michal Hruby (mhr3) wrote :

Awesometastic!

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