Re: [Tracker] Review request: miner-web-review branch

[Martyn Russell <martyn lanedo com>]

On 01/03/10 11:31, Adrien Bustany wrote:
> Hello everyone,

Hej Adrien,

> The miner-web-review branch contains a patch that adds a new class in
> libtracker-miner, called TrackerMinerWeb. This is basically an abstract
> subclass of TrackerMiner which adds a few signals and conveniences
> useful when writing miners fetching data from web services.
>
> This is only the first step of the review, in the pipe are:
>
> - Various web miners. But I want the API to be freezed before porting them.
> - A miner manager used to bind miners to their web services. This one is
> not fully cooked yet, and Rob Taylor suggested it should be a gnome
> control panel applet more than a tracker-preferences pane. Any feedback
> is welcome.
>
> The earthquake here might have slipped in a few bugs in my code, so
> review carefuly ;)

Glad to hear you're OK. I will try to find time to review this branch 
this week.

--
Regards,
Martyn