* David MENTRE <[email protected]> [2010-06-01 17:32:08]:

> Hi Maxime,
> 
> 2010/6/1 Maxime Petazzoni <[email protected]>:
> > Note that I didn't add/change this. It's barely a duplicate of the
> > already existing rendering_already_exists code that looked for
> > successful renderings by administrative boundary and which already used
> > status==2 in its query.
> 
> OK. So all that status==2 code is bad. :)

Yep. But we can fix that later :-)

> > Other than that, is the patch ok for merging?
> 
> I think so, but my opinion is rather lightweight considering my current
> involvement in maposmatic.

Ok. Since no one else objected, I took the liberty of integrating the
changes and pushed them. I'll deploy to dev.maposmatic.org during the
week-end for some more testing along with some other fixes and i18n
updates.

- Maxime
-- 
Maxime Petazzoni <http://www.bulix.org>
 ``One by one, the penguins took away my sanity.''
Linux kernel and software developer at MontaVista Software

Attachment: signature.asc
Description: Digital signature

Reply via email to