[Bug 41772] TP Logger: Introduce TplEventWalker
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Tue Jul 10 05:34:43 CEST 2012
https://bugs.freedesktop.org/show_bug.cgi?id=41772
--- Comment #7 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk> 2012-07-09 20:34:43 PDT ---
(In reply to comment #6)
> (In reply to comment #5)
> > I did an API only review.The log manager part seems fine to me, other method
> > can be added later. For the TplLogWalker though I'd do some changes.
>
> Thank you for the review!
>
> I am, personally, a bit confused about what is forward and what is backward.
> eg., if we have 100 events, with 0 being the oldest and 99 being the most
> recent, then currently we return 95-99 in the first batch, 90-94 in the second
> batch and so on? Is this forward or backward?
Good point, as this return stuff from most recent, going forward would be
getting older data. But that could be confusing.
The idea is that I don't want UI to keep all the data into RAM. So UIs need a
way to forget some of the information, and request it later (in an efficient
way).
--
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
You are the assignee for the bug.
More information about the telepathy-bugs
mailing list