kyle.tk <[EMAIL PROTECTED]> wrote:
   ...
>       new_mac = ''
>       c = 0
>       while len(new_mac) < 16:
>               new_mac += mac[c:c+2] + ':'
>               c=c+2
>       return new_mac[:-1].upper()
   ...
> The part I think is bad is the while loop part. Could it be better?

What about replacing this whole block with, say...:

return ':'.join(mac[c:c+2] for c in range(0, 12, 2)).upper()

(as you did already check that len(mac) is 12...)


Alex
-- 
http://mail.python.org/mailman/listinfo/python-list

Reply via email to