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