Re: [Maposmatic-dev] [PATCH] [ocitysmap] Rescale category header where needed

[David MENTRE]

Hello Jeroen,

Just a few comments...

2012/4/17 Jeroen van Rijn <address@hidden>:
> +        # Measure header text and rescale if needed

I would put more hash marks (###) to separate more clearly this part
of the code. In fact, maybe this should be put in a specific
_rescale_something function. I don't know.

> +        if scale < 1.2:

How has been determined this 1.2 value?

> +        # Restore target width
> +        layout.set_width(target_width)
> +        layout.context_changed()
> +        # Restore context
> +        ctx.restore()

Is it really necessary to call set_width(target_width) as the context
is restored just afterwards?

Overall, very cleanly commented code, nice to read!

Best regards,
d.