[sugar] [PATCH] Trac #7480: Need to 'reset' the network configurations - short term fix

[Michael Stone]

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