On 02/27/2012 06:31 PM, Martin Kosek wrote:


4) Minor change:
-                except Exception:
+                except:


Don't do that. It would for example disable Ctrl+C by trapping KeyboardInterrupt.

PEP8 has a paragraph on this, search for 'except Exception:'


--
PetrĀ³

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to