[Thuban-commits] r2891 - trunk/thuban/Thuban/UI

[Bjoern Ricks]

On Thursday 14 July 2011 13:15:14 Bernhard Herzog wrote:
> This is better written as
>
>        if not bbox:
>            #There are no layers so we have nothing to scale
>            return scale, scale
>
>        llx, lly, urx, ury = bbox
>
> The rest of the method after the if-statement only makes sense if there is
> a bbox, so you'd either need to put the entire rest of the method into the
> else-clause or simply omit the else as above because the early return if
> there's no bbox essentially turns everything after the if statement into an
> else clause anyway.
>
>   Bernhard
Of course else doesn't make sense here. Just removed it and did another 
commit. Thanks!

Björn
-- 
Bjoern Ricks |  ++49-541-335083-167  |  http://www.intevation.de/
Intevation GmbH, Neuer Graben 17, 49074 Osnabrück | AG Osnabrück, HR B 18998
Geschäftsführer: Frank Koormann, Bernhard Reiter, Dr. Jan-Oliver Wagner
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
Url : http://www.intevation.de/pipermail/thuban-devel/attachments/20110714/6b4daed4/attachment.bin