> I have never seen it mentioned in PEP008 but it is a good form to put
> the inner function definitions at the beginning of the enclosing
> function, to ease reading later.

Reading is easier when









related things are close!

>> +            if limit: url = {
>> +                'mirror': misc.to_utf8(url),
>> +                'kwargs': { 'max_connections': limit },
>> +            }

> Hm, and the formatting here looks odd. 'url' on a newline?

ACK.  Didn't like double indent here, but on the second
thought.. yes it is ugly.
_______________________________________________
Yum-devel mailing list
[email protected]
http://lists.baseurl.org/mailman/listinfo/yum-devel

Reply via email to