http://bugzilla.moblin.org/show_bug.cgi?id=8049





--- Comment #19 from pohly <[email protected]>  2009-12-03 02:20:32 PST ---
(In reply to comment #18)
> (In reply to comment #17)
> > (In reply to comment #16)
> > > There are 2 issues left to be implemented:
> > > 1) keys in reports: there should be one more info to be added into the 
> > > keys
> > >    peer name is put in the header of the key string, e.g., 
> > > [peername]-[current
> > > key].
> > 
> > Why put it into the key and not some value? It makes all keys longer. We are
> > not mixing values from different reports into the same hash, so adding a 
> > single
> > key like "peer" is sufficient.
> The peer name is not known for this case. So we have 2 choices:
> 1) write 'peer' in the status.ini file
> 2) parse the peer name from the directory
> I prefer the second one for we have sort the directory list before reading the
> content. If so, we have to escape some characters for peer names.

Option 2 was also what I was thinking of as the source for the peer name. You
are right about having to escape the peer names.

> > > 2) return reports sorted by time for all reports belonging to different 
> > > peers
> > >    if peer name is empty, all reports are sorted by their creation time 
> > > and
> > > then picked up according to "start" and "count".
> > 
> > Okay.
> Since we don't know the creation time of the file/directory, we have to 
> extract
> the creation time from the directory time and then sort directory list.

With "directory time", do you mean the date+time embedded in the directory
name? If so, then I agree.

-- 
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
_______________________________________________
Syncevolution-issues mailing list
[email protected]
http://lists.syncevolution.org/listinfo/syncevolution-issues

Reply via email to