Re: [Tracker] REVIEW: external-crawler branch, part2

2014-08-08 Thread Philip Van Hoof
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 87c30cc73629ff54ff8e5170ded499c2cbbda61a About TrackerEnumeratorIface: In my opinion an enumerator has a Next() method to enumerate to the next element. I think it's a bit wierd to return a linked list on Start() of the enumerator API. You can find

Re: [Tracker] REVIEW: external-crawler branch, part2

2014-08-08 Thread Philip Van Hoof
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 8/08/2014 10:09, Philip Van Hoof wrote: > 87c30cc73629ff54ff8e5170ded499c2cbbda61a Sorry it's like this instead of course: > Enumerable thingThatHasItems = new List(); Enumerable e = > thingThatHasItems.Enumerate(); // (not this but) Enumerator e

Re: [Tracker] REVIEW: external-crawler branch, part2

2014-08-08 Thread Martyn Russell
On 08/08/14 09:09, Philip Van Hoof wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 87c30cc73629ff54ff8e5170ded499c2cbbda61a About TrackerEnumeratorIface: In my opinion an enumerator has a Next() method to enumerate to the next element. I think it's a bit wierd to return a linked list on S