On Mon, Aug 11, 2008 at 09:31:26PM -0400, Erik Garrison wrote:
>Attached is a patch which adds a 'reset network configuration' button to
>the network tab of the sugar control panel.
Thanks very much for the patch, and for the thoughtful design.
>+ n = 1 + max(map(lambda x: int(x.replace('networks.cfg.bak.', '')),
>+ network_cfgs))
This expression, while cute, is unsafe and will result in failure if you
ever encounter a file with a name like 'networks.cfg.bak.hi'. Please
write total code or handle exceptions appropriately.
(Incidentally, does anything bad happen if your code does happen to
generate exceptions?)
Regards,
Michael
_______________________________________________
Sugar mailing list
Sugar@lists.laptop.org
http://lists.laptop.org/listinfo/sugar
